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
Fix finished_mutations_to_keep=0 for MergeTree (as docs says 0 is to keep everything) #60031
Fix finished_mutations_to_keep=0 for MergeTree (as docs says 0 is to keep everything) #60031
Conversation
This is an automated comment for commit 3c08e19 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
|
e60cf0b
to
adac7f5
Compare
adac7f5
to
89eb2a3
Compare
|
89eb2a3
to
89030ae
Compare
There are a lot of noise regarding to unrelated issues (e.g. the io_uring stuff), so let merge a master when you want to merge to trigger a new run. This change is simple, but might affect important parts, so it is better to have a another run in my opinion. |
89030ae
to
f04cd72
Compare
tests/queries/0_stateless/02994_merge_tree_mutations_cleanup.sql.j2
Outdated
Show resolved
Hide resolved
So, is there anything else left? UPD: will do a rebase to fix CI |
…keep everything) Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
…dMergeTree) Even if this should be done, only related mutations should be cleaned, not all. Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
…is_dropped Since after DROP PART mutations are not cleaned anymore. Here an example: objects_at_the_end = list_objects(cluster) > assert objects_at_the_end == objects_empty_table E AssertionError: assert ['data/evt/iczupcswcatzvjikqwmovahturdht', 'data/oxq/dmkgqbdzeeiwmukogoxawfoxxnrqs'] == ['data/oxq/dmkgqbdzeeiwmukogoxawfoxxnrqs'] E At index 0 diff: 'data/evt/iczupcswcatzvjikqwmovahturdht' != 'data/oxq/dmkgqbdzeeiwmukogoxawfoxxnrqs' E Left contains one more item: 'data/oxq/dmkgqbdzeeiwmukogoxawfoxxnrqs' E Full diff: E [ E + 'data/evt/iczupcswcatzvjikqwmovahturdht', E 'data/oxq/dmkgqbdzeeiwmukogoxawfoxxnrqs', E ] test_replicated_zero_copy_projection_mutation/test.py:155: AssertionError And decoded paths: node1 :) select local_path from system.blob_storage_log where remote_path = 'data/evt/iczupcswcatzvjikqwmovahturdht' SELECT local_path FROM system.blob_storage_log WHERE remote_path = 'data/evt/iczupcswcatzvjikqwmovahturdht' Query id: 9ee5a9c0-c3b7-46ad-82bd-64c8bcbda78d ┌─local_path────────────────────────────────────────────────────────┐ │ store/bce/bcea71c9-35cd-4368-9504-c563253b1964/tmp_mutation_1.txt │ │ store/bce/bcea71c9-35cd-4368-9504-c563253b1964/mutation_1.txt │ └───────────────────────────────────────────────────────────────────┘ Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
Otherwise it fails in private fork CI Co-authored-by: János Benjamin Antal <antaljanosbenjamin@users.noreply.github.com>
72d92b0
to
3c08e19
Compare
This PR looks okay, I will merge it soon. Sorry for not merging yet, but our CI started to show strange errors and I was worried to merge things without figuring those things out first. |
Unit tests failed with:
I doubt this is connected to this PR. I will open an issue about this.
|
9cd7e61
into
ClickHouse:master
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix finished_mutations_to_keep=0 for MergeTree (as docs says 0 is to keep everything)