Skip to content

Fix incorrect parent_part_offset calculation in unordered MergeTree during merge#81188

Merged
Avogar merged 1 commit intoClickHouse:masterfrom
amosbird:fix-parent-part-offset-in-unordered-merge-tree
Jun 3, 2025
Merged

Fix incorrect parent_part_offset calculation in unordered MergeTree during merge#81188
Avogar merged 1 commit intoClickHouse:masterfrom
amosbird:fix-parent-part-offset-in-unordered-merge-tree

Conversation

@amosbird
Copy link
Copy Markdown
Collaborator

@amosbird amosbird commented Jun 3, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

Fix incorrect parent_part_offset calculation in unordered MergeTree during merge. This addresses #78429 (comment) . Mark as not for changelog, as this is part of ongoing work on projection index which is not landed yet.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jun 3, 2025

Workflow [PR], commit [247c5f3]

@clickhouse-gh clickhouse-gh bot added pr-not-for-changelog This PR should not be mentioned in the changelog submodule changed At least one submodule changed in this PR. labels Jun 3, 2025
@amosbird amosbird force-pushed the fix-parent-part-offset-in-unordered-merge-tree branch from ffa5e00 to 247c5f3 Compare June 3, 2025 03:59
@amosbird amosbird removed the submodule changed At least one submodule changed in this PR. label Jun 3, 2025
@Avogar Avogar self-assigned this Jun 3, 2025
@Avogar
Copy link
Copy Markdown
Member

Avogar commented Jun 3, 2025

Stateless tests (debug) - #80288
CH Inc sync - known issue with not found key.

@Avogar Avogar added this pull request to the merge queue Jun 3, 2025
@amosbird
Copy link
Copy Markdown
Collaborator Author

amosbird commented Jun 3, 2025

Wow, that was fast. Thanks for the swift review!

Merged via the queue into ClickHouse:master with commit 9e58cb2 Jun 3, 2025
118 of 121 checks passed
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog 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.

3 participants