Skip to content

[core] Fix blob read failure after alter table and compaction#7618

Merged
JingsongLi merged 8 commits intoapache:masterfrom
littlecoder04:blob_fix
Apr 10, 2026
Merged

[core] Fix blob read failure after alter table and compaction#7618
JingsongLi merged 8 commits intoapache:masterfrom
littlecoder04:blob_fix

Conversation

@littlecoder04
Copy link
Copy Markdown
Contributor

@littlecoder04 littlecoder04 commented Apr 9, 2026

Purpose

Reading a blob table fails after ALTER TABLE SET and compaction with:
java.lang.IllegalArgumentException: All files in this bunch should have the same schema id. This PR fixes the above issue by do not check schemaId for blob file and do not allow rename for blob col.

Tests

testReadBlobAfterAlterTableAndCompaction
testReadBlobAfterAddColumnAndCompaction
testReadBlobAfterDropColumnAndCompaction
testRenameBlobColumnReadFailure: this is used to reproduce the issue if rename for blob is allowed

@littlecoder04 littlecoder04 marked this pull request as draft April 9, 2026 15:48
@littlecoder04 littlecoder04 changed the title [core] Fix blob read failure after ALTER TABLE [core] Fix blob read failure after alter table Apr 9, 2026
@littlecoder04 littlecoder04 changed the title [core] Fix blob read failure after alter table [core] Fix blob read failure after alter table and compaction Apr 10, 2026
@littlecoder04 littlecoder04 marked this pull request as ready for review April 10, 2026 02:56
@littlecoder04 littlecoder04 marked this pull request as draft April 10, 2026 03:07
@littlecoder04 littlecoder04 marked this pull request as ready for review April 10, 2026 03:13
Copy link
Copy Markdown
Contributor

@leaves12138 leaves12138 left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

+1

@JingsongLi JingsongLi merged commit 72600f9 into apache:master Apr 10, 2026
17 checks passed
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.

3 participants