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
MergeTree mutations reuse source part index granularity #57352
MergeTree mutations reuse source part index granularity #57352
Conversation
This is an automated comment for commit 6578fe5 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
|
I cannot reproduce issue without my data:
Insert data from broken part:
Add and materialize index:
In parts index is materialized:
Materialize index once again (amount of granules in part changes):
In system.parts marks size changed in this part:
Check system.data_skipping_indices table, index looks broken:
After than each next index materialization will change marks size size:
But index is stil broken:
CompactPart writer logs during this process: Create table and insert data from file. We make single write with 112618 block size:
Add index and materialize it:
Materialize index once again:
Materialize index once again:
Materialize index once again:
|
It can be related #16925.
|
After fix: Create table and insert data from file. We make single write with 112618 block size:
Materialize index once again:
Materialize index once again:
|
0bd9349
to
f0e53fd
Compare
…55b645c33e88451ec6244144f7434 Cherry pick #57352 to 23.3: MergeTree mutations reuse source part index granularity
…55b645c33e88451ec6244144f7434 Cherry pick #57352 to 23.8: MergeTree mutations reuse source part index granularity
…55b645c33e88451ec6244144f7434 Cherry pick #57352 to 23.9: MergeTree mutations reuse source part index granularity
…d55b645c33e88451ec6244144f7434 Cherry pick #57352 to 23.10: MergeTree mutations reuse source part index granularity
Backport #57352 to 23.3: MergeTree mutations reuse source part index granularity
@CurtizJ, you have merged it without a test:
This is not allowed. |
Backport #57352 to 23.10: MergeTree mutations reuse source part index granularity
Backport #57352 to 23.8: MergeTree mutations reuse source part index granularity
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix possible broken skipping indexes after materialization in MergeTree compact parts.