SigNoz / signoz-otel-collector

SigNoz distro for OpenTelemetry Collector
44 stars 39 forks source link

chore: scrape normalized_query_hash as string value instead of uint64 from system.query_log table #320

Closed raj-k-singh closed 3 months ago

raj-k-singh commented 3 months ago

Right now the clickhousesystemtablesreceiver scrapes normalized_query_hash column from system.query_log table as a uint64 value, but the value ends up in int64 fields which leads to integer overflow (the logs table supports only int64 values and pdata also stores uints as ints )

Since the query hash is used only for equality comparisons, it should be scraped as a string value to preserve the original value

fixes https://github.com/SigNoz/signoz-otel-collector/issues/319