influx_inspect buildtsi panics on mmapped TSM file - Githubissues

influxdata / influxdb

Scalable datastore for metrics, events, and real-time analytics

https://influxdata.com

Apache License 2.0

28.57k stars 3.53k forks source link

influx_inspect buildtsi panics on mmapped TSM file #24994

Open davidby-influx opened 4 months ago

davidby-influx commented 4 months ago

Stack:

https://github.com/influxdata/influxdb/blob/79ae559f40ec7cad3311ac458f61a8d8cfd49417/tsdb/series_file.go#L344 https://github.com/influxdata/influxdb/blob/79ae559f40ec7cad3311ac458f61a8d8cfd49417/tsdb/index/tsi1/log_file.go#L697 https://github.com/influxdata/influxdb/blob/79ae559f40ec7cad3311ac458f61a8d8cfd49417/tsdb/index/tsi1/log_file.go#L658 https://github.com/influxdata/influxdb/blob/79ae559f40ec7cad3311ac458f61a8d8cfd49417/tsdb/index/tsi1/log_file.go#L564 https://github.com/influxdata/influxdb/blob/79ae559f40ec7cad3311ac458f61a8d8cfd49417/tsdb/index/tsi1/partition.go#L709 https://github.com/influxdata/influxdb/blob/79ae559f40ec7cad3311ac458f61a8d8cfd49417/tsdb/index/tsi1/index.go#L679

davidby-influx commented 3 months ago

Limiting influx_inspectconcurrency to 12 allows it to complete successfully on the customer installation.

davidby-influx commented 3 months ago

Whereas respecting the errors returned parsing the measurement names still panics:

https://github.com/influxdata/influxdb/pull/24995