Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

influx_inspect buildtsi panics on mmapped TSM file #24994

Open
davidby-influx opened this issue May 10, 2024 · 2 comments
Open

influx_inspect buildtsi panics on mmapped TSM file #24994

davidby-influx opened this issue May 10, 2024 · 2 comments

Comments

@davidby-influx
Copy link
Contributor

image

Stack:

dst = append(dst, name...)

seriesKey = tsdb.AppendSeriesKey(f.keyBuf[:0], e.name, e.tags)

f.execSeriesEntry(e)

f.execEntry(entry)

ids, err := p.activeLogFile.AddSeriesList(p.seriesIDSet, names, tagsSlice, tracker)

ids, err := i.partitions[idx].createSeriesListIfNotExists(pNames[idx], pTags[idx], tracker)

@davidby-influx davidby-influx self-assigned this May 10, 2024
@davidby-influx davidby-influx changed the title influx_inspect buildtsi panics on mmaped TSM file influx_inspect buildtsi panics on mmapped TSM file May 10, 2024
@davidby-influx
Copy link
Contributor Author

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

@davidby-influx
Copy link
Contributor Author

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

#24995

@davidby-influx davidby-influx closed this as not planned Won't fix, can't repro, duplicate, stale May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant