Skip to content

Fix logical error in full_sorting_merge join with duplicate columns#89495

Merged
vdimir merged 3 commits intomasterfrom
vdimir/join_dup_columns_bug
Nov 11, 2025
Merged

Fix logical error in full_sorting_merge join with duplicate columns#89495
vdimir merged 3 commits intomasterfrom
vdimir/join_dup_columns_bug

Conversation

@vdimir
Copy link
Copy Markdown
Member

@vdimir vdimir commented Nov 4, 2025

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

Fix logical error in full_sorting_merge join with duplicate columns. Resolves #86957

Details

Resolves #86957

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Nov 4, 2025

Workflow [PR], commit [3f5f778]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Nov 4, 2025
@vdimir vdimir force-pushed the vdimir/join_dup_columns_bug branch from ddc50b0 to 8e3f952 Compare November 4, 2025 17:32
@SmitaRKulkarni SmitaRKulkarni self-assigned this Nov 4, 2025
@vdimir vdimir force-pushed the vdimir/join_dup_columns_bug branch from 8e3f952 to 0bd3cf4 Compare November 5, 2025 10:14
Copy link
Copy Markdown
Member

@SmitaRKulkarni SmitaRKulkarni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vdimir vdimir force-pushed the vdimir/join_dup_columns_bug branch from 5f0bad2 to 3f5f778 Compare November 10, 2025 11:56
@vdimir vdimir enabled auto-merge November 11, 2025 14:14
@vdimir vdimir added this pull request to the merge queue Nov 11, 2025
Merged via the queue into master with commit a2731b0 Nov 11, 2025
131 checks passed
@vdimir vdimir deleted the vdimir/join_dup_columns_bug branch November 11, 2025 16:31
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Nov 11, 2025
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.

Logical error: Invalid number of columns in chunk pushed to OutputPort with full_sorting_merge

3 participants