Skip to content

[branch-52] fix: Ensure columns are casted to the correct names with Unions (#20146)#20879

Open
alamb wants to merge 2 commits intoapache:branch-52from
alamb:alamb/backport_20146
Open

[branch-52] fix: Ensure columns are casted to the correct names with Unions (#20146)#20879
alamb wants to merge 2 commits intoapache:branch-52from
alamb:alamb/backport_20146

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Mar 11, 2026

nuno-faria and others added 2 commits March 11, 2026 15:44
@nuno-faria @alamb
fix: Ensure columns are casted to the correct names with Unions (apac…
b212a81 
…he#20146)

- Closes apache#20123.

When aliasing a cast, use the original name to prevent duplicate column
names errors.

- Updated `coerce_exprs_for_schema` to use the correct name.
- Updated tests.

Yes.

No.
@alamb
test: add missing duplicate_name_in_union substrait fixture
b534dfa
@github-actions github-actions bot added logical-expr Logical plan and expressions optimizer Optimizer rules substrait Changes to the substrait crate labels Mar 11, 2026
@alamb alamb mentioned this pull request Mar 11, 2026
Open
9 tasks
nuno-faria
nuno-faria approved these changes Mar 11, 2026
View reviewed changes
Copy link
Contributor

@nuno-faria nuno-faria left a comment

Choose a reason for hiding this comment

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

Thanks @alamb.

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

Reviewers

@nuno-faria nuno-faria nuno-faria approved these changes

Assignees

No one assigned

Labels

logical-expr Logical plan and expressions optimizer Optimizer rules substrait Changes to the substrait crate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alamb @nuno-faria