Skip to content

Flink: Fix ALTER TABLE to add column to specific position#16419

Merged
pvary merged 2 commits into
apache:mainfrom
SteveStevenpoor:add-column-to-pos-bug
May 20, 2026
Merged

Flink: Fix ALTER TABLE to add column to specific position#16419
pvary merged 2 commits into
apache:mainfrom
SteveStevenpoor:add-column-to-pos-bug

Conversation

@SteveStevenpoor
Copy link
Copy Markdown
Contributor

@SteveStevenpoor SteveStevenpoor commented May 19, 2026

This closes #16420.

Summary

  • Modified FlinkAlterTableUtil to fix the bug
  • Added testAlterTableAddColumnPosition() to TestFlinkCatalogTable to verify changes


@TestTemplate
public void testAlterColumnPositionAfter() {
// first add then modify position
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These are for a different PR

@pvary pvary merged commit 684a187 into apache:main May 20, 2026
29 checks passed
@pvary pvary changed the title Flink: Add column to position bug Flink: Fix ALTER TABLE to add column to specific position May 20, 2026
@pvary
Copy link
Copy Markdown
Contributor

pvary commented May 20, 2026

Merged to main.
Thanks @SteveStevenpoor for the fix.

Please backport to the other Flink branches:

g d HEAD^ HEAD flink/v2.1 | sed "s/v2.1/v2.0/g">/tmp/patch;g apply -3 -p1 /tmp/patch   
g d HEAD^ HEAD flink/v2.1 | sed "s/v2.1/v1.20/g">/tmp/patch;g apply -3 -p1 /tmp/patch  

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flink: Add column to position bug

2 participants