Skip to content

[server][bug] Fix OOB in partial update/delete after ADD COLUMN#3000

Merged
wuchong merged 2 commits into
apache:mainfrom
fresh-borzoni:fix-partial-update-schema-evolution
Apr 8, 2026
Merged

[server][bug] Fix OOB in partial update/delete after ADD COLUMN#3000
wuchong merged 2 commits into
apache:mainfrom
fresh-borzoni:fix-partial-update-schema-evolution

Conversation

@fresh-borzoni
Copy link
Copy Markdown
Member

Summary

closes #2988

When a table has partial updates enabled and ADD COLUMN is performed, reading old rows from RocksDB with fewer fields than the latest schema causes ArrayIndexOutOfBoundsException in PartialUpdater.

Fix by treating missing fields as null

@fresh-borzoni
Copy link
Copy Markdown
Member Author

@wuchong @polyzos PTAL 🙏

@fresh-borzoni fresh-borzoni force-pushed the fix-partial-update-schema-evolution branch from 3b7f793 to 72104f8 Compare April 4, 2026 15:30
@fresh-borzoni fresh-borzoni force-pushed the fix-partial-update-schema-evolution branch from 72104f8 to cd6d329 Compare April 4, 2026 15:48
Copy link
Copy Markdown
Member

@wuchong wuchong left a comment

Choose a reason for hiding this comment

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

LGTM.

I added an inline comment to explain the changed code.

@wuchong wuchong merged commit 8d757da into apache:main Apr 8, 2026
18 of 21 checks passed
fresh-borzoni added a commit to fresh-borzoni/fluss that referenced this pull request Apr 16, 2026
Ugbot pushed a commit to Ugbot/fluss that referenced this pull request Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ArrayIndexOutOfBoundsException in partial updates after ADD COLUMN

2 participants