Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform #88090
Merged
shankar-iyer merged 6 commits intoClickHouse:masterfrom Oct 9, 2025
Merged
Conversation
Contributor
|
Workflow [PR], commit [de19bf5] Summary: ❌
|
clickhouse-gh
bot
added
the
pr-performance
| @@ -0,0 +1,72 @@ | |||
| -- Test for https://github.com/ClickHouse/ClickHouse/pull/76978 | |||
Member
There was a problem hiding this comment.
@shankar-iyer Maybe makes sense to add performance test? What do you think?
Member
Author
There was a problem hiding this comment.
ack, will do in a follow on PR. Test replacing_final_non_intersecting.xml looks to be a good reference.
Member
Author
|
Previous CI run was 100% green! But latest CI run has some integration test failures, verified that they are unrelated to the PR ( |
Merged
via the queue into
ClickHouse:master
with commit Oct 9, 2025
b58d5e0
118 of 123 checks passed
robot-ch-test-poll4
added
the
pr-synced-to-cloud
mkmkme
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Nov 6, 2025
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
25 tasks
zvonand
added a commit
to Altinity/ClickHouse
that referenced
this pull request
Nov 7, 2025
25.3.8 Backport of ClickHouse#88090 - Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
zvonand
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Dec 8, 2025
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
25 tasks
zvonand
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Dec 15, 2025
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
zvonand
added a commit
to Altinity/ClickHouse
that referenced
this pull request
Dec 16, 2025
25.8.12 Backport of ClickHouse#88090: Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
zvonand
added a commit
to Altinity/ClickHouse
that referenced
this pull request
Dec 23, 2025
25.8.12 Backport of ClickHouse#88090: Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
25 tasks
zvonand
added a commit
to Altinity/ClickHouse
that referenced
this pull request
Dec 24, 2025
25.8.13 Backport of ClickHouse#88090: Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
zvonand
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Jan 15, 2026
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
25 tasks
zvonand
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Jan 22, 2026
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
25 tasks
zvonand
added a commit
to Altinity/ClickHouse
that referenced
this pull request
Jan 28, 2026
25.8.15 Backport of ClickHouse#88090: Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
zvonand
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Feb 5, 2026
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
25 tasks
zvonand
added a commit
to Altinity/ClickHouse
that referenced
this pull request
Feb 10, 2026
24.8.14 Backport of ClickHouse#88090: Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
zvonand
pushed a commit
to Altinity/ClickHouse
that referenced
this pull request
Mar 17, 2026
…with_filter Optimize ReplacingMergeTree is_deleted FINAL queries by adding a filter expression transform
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resolves #65180 and #69164.
Currently, all ranges in a query with
FINALclause on aReplacingMergeTreetable withis_deletedcolumn have to be processed by FINAL merge transform.Add a
FilterTransformcorrespdoning tois_deleted = 0for parts & ranges thus enabling parallel processing and skipFINALMerge transform. The parts & ranges that qualify are 1) Partitions with only a singlepartwhendo_not_merge_across_partitions_select_finalisTrue2) Non-intersecting ranges identified by the existing split ranges into intersecting/non-intersecting optimization.Performance example : Link
Changelog category (leave one):
Changelog entry (a [user-readable short description]
FINALclause on.aReplacingMergeTreetable with theis_deletedcolumn now executes faster because of improved parallelization from 2 existing optimizations : 1)do_not_merge_across_partitions_select_finaloptimization for partitions of the table that have only a singlepart2) Split other selected ranges of the table intointersecting / non-intersectingand only intersecting ranges have to pass through FINAL merging transform.