Unblock ttl part drops for cold volumes by Michicosun · Pull Request #90059 · ClickHouse/ClickHouse · GitHub
Skip to content

Unblock ttl part drops for cold volumes#90059

Merged
Michicosun merged 6 commits into
masterfrom
unblock_ttl_drop_merges_in_cold_volumes
Nov 21, 2025
Merged

Unblock ttl part drops for cold volumes#90059
Michicosun merged 6 commits into
masterfrom
unblock_ttl_drop_merges_in_cold_volumes

Conversation

@Michicosun

@Michicosun Michicosun commented Nov 14, 2025

Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Split part ranges by volume characteristics to enable TTL drop merges for cold volumes. After this patch, parts with a max TTL < now will be removed from cold storage. The algorithm will schedule only single part drops.

Fixes #85636.

@Michicosun Michicosun marked this pull request as ready for review November 14, 2025 15:38
@clickhouse-gh

clickhouse-gh Bot commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-bugfix Pull request with bugfix, not backported by default label Nov 14, 2025
@CurtizJ CurtizJ self-assigned this Nov 14, 2025
@Michicosun Michicosun changed the title Unblock ttl drop merges in cold volumes Unblock ttl part drops for cold volumes Nov 14, 2025
@Michicosun

Copy link
Copy Markdown
Member Author

Needs #90077

@Michicosun Michicosun added this pull request to the merge queue Nov 21, 2025
Merged via the queue into master with commit 77bf5e0 Nov 21, 2025
126 of 130 checks passed
@Michicosun Michicosun deleted the unblock_ttl_drop_merges_in_cold_volumes branch November 21, 2025 15:48
@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Nov 21, 2025
@aalexfvk

Copy link
Copy Markdown
Contributor

@Michicosun What do you think about backporting this ?

@Michicosun

Copy link
Copy Markdown
Member Author

@aalexfvk What versions do you use?

@aalexfvk

Copy link
Copy Markdown
Contributor

@aalexfvk What versions do you use?

All starting from 25.3

zvonand pushed a commit to Altinity/ClickHouse that referenced this pull request Feb 3, 2026
…merges_in_cold_volumes

Unblock ttl part drops for cold volumes
zvonand pushed a commit to Altinity/ClickHouse that referenced this pull request Feb 5, 2026
…merges_in_cold_volumes

Unblock ttl part drops for cold volumes
MyroTk added a commit to Altinity/ClickHouse that referenced this pull request Feb 6, 2026
25.8.16 Stable Backport of ClickHouse#90059: Unblock ttl part drops for cold volumes
@aalexfvk

Copy link
Copy Markdown
Contributor

@Michicosun
Hello! Just a reminder about the backport.

@Michicosun

Copy link
Copy Markdown
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TTLDelete + ttl_only_drop_parts does not work for volumes with prefer_not_to_merge=true

5 participants