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
Reuse granule during skip index reading #55683
Conversation
This is an automated comment for commit 602f01f with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
bfa7fca
to
7637b1e
Compare
93acac6
to
f35d10f
Compare
Performance improvement is significant
Nothing obvious in all logs. It seems the fuzzer itself aborted.
test_postgresql_replica_database_engine_2 FAIL. Unrelated.
test_dictionaries_redis FAIL. Unrelated.
test_postgresql_replica_database_engine_2 FAIL. Unrelated. |
We can introduce a new min-max index similar to https://sneller.ai/blog/sneller-time-index/ . It will have adaptive granularity and can be configured to always live in memory similar to primary key. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Avoid unnecessary reconstruction of index granules when reading skip indexes. This addresses #55653 (comment) .