Skip to content

Avoid UB in distributed index analysis on fallback to local replica and there are more then max_parallel_replicas replicas#95263

Merged
azat merged 6 commits intomasterfrom
dist-index-analysis-local-replica-fix
Jan 27, 2026
Merged

Avoid UB in distributed index analysis on fallback to local replica and there are more then max_parallel_replicas replicas#95263
azat merged 6 commits intomasterfrom
dist-index-analysis-local-replica-fix

Conversation

@azat
Copy link
Member

@azat azat commented Jan 27, 2026

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):

Avoid UB in distributed index analysis on fallback to local replica and there are more then max_parallel_replicas replicas

Fixes: #95222
Closes: #95234

@azat azat requested a review from KochetovNicolai January 27, 2026 06:44
@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Jan 27, 2026

Workflow [PR], commit [9928d96]

Summary:

job_name test_name status info comment
AST fuzzer (amd_ubsan) failure
Logical error: Expected the argument №A ('B' of type C) to have D rows, but it has E (STID: 2959-4e38) FAIL cidb
Performance Comparison (amd_release, master_head, 1/6) failure
Check Results failure

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Jan 27, 2026
@azat azat added this pull request to the merge queue Jan 27, 2026
Merged via the queue into master with commit 78de9a4 Jan 27, 2026
131 of 134 checks passed
@azat azat deleted the dist-index-analysis-local-replica-fix branch January 27, 2026 19:36
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jan 27, 2026
robot-clickhouse-ci-2 added a commit that referenced this pull request Jan 27, 2026
Cherry pick #95263 to 26.1: Avoid UB in distributed index analysis on fallback to local replica and there are more then max_parallel_replicas replicas
robot-clickhouse added a commit that referenced this pull request Jan 27, 2026
…llback to local replica and there are more then max_parallel_replicas replicas
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Jan 27, 2026
@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jan 27, 2026
clickhouse-gh bot added a commit that referenced this pull request Jan 27, 2026
Backport #95263 to 26.1: Avoid UB in distributed index analysis on fallback to local replica and there are more then max_parallel_replicas replicas
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo v26.1-must-backport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AddressSanitizer (STID: 3785-4c97)

6 participants