-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Add table function mergeTreeIndex
#58140
Conversation
This is an automated comment for commit 5330824 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
|
auto mark = marks_loader->getMark(i, col_idx); | ||
|
||
compressed_data[i] = mark.offset_in_compressed_file; | ||
uncompressed_data[i] = mark.offset_in_decompressed_block; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And row count for adaptive granularity?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's already added (column rows_in_granule
)
7f9fef9
to
482e01d
Compare
Test failures are unrelated. |
However, there's another critical issue introduced by this PR, and I failed to fix it quickly, so I have to revert, sorry :( |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Added table function
mergeTreeIndex
. It represents the contents of index and marks files ofMergeTree
tables. It can be used for introspection. Syntax:mergeTreeIndex(database, table, [with_marks = true])
wheredatabase.table
is an existing table withMergeTree
engine.Documentation entry for user-facing changes