Skip to content

[core] Support overwriting multiple specified partitions#8789

Merged
JingsongLi merged 3 commits into
apache:masterfrom
tsreaper:feature/overwrite-many-partitions
Jul 23, 2026
Merged

[core] Support overwriting multiple specified partitions#8789
JingsongLi merged 3 commits into
apache:masterfrom
tsreaper:feature/overwrite-many-partitions

Conversation

@tsreaper

Copy link
Copy Markdown
Contributor

Purpose

Currently, the commit API only supports overwriting a single partition. This PR adds an interface withOverwrite(List<BinaryRow>) to support overwriting multiple specified partitions.

Tests

  • OverwriteTableTest.

@tsreaper
tsreaper force-pushed the feature/overwrite-many-partitions branch 2 times, most recently from 7893026 to 972fbbf Compare July 22, 2026 08:25
@JingsongLi

Copy link
Copy Markdown
Contributor

After adding an overloaded method with the same name, List<BinaryRow>, the previously valid withOverwrite(null) now causes a compilation ambiguity; I confirmed that this issue can be reproduced using javac on the current head.

The method name could be withOverwrites.

@tsreaper
tsreaper force-pushed the feature/overwrite-many-partitions branch from 972fbbf to 3db5e62 Compare July 22, 2026 10:39

@JingsongLi JingsongLi left a comment

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.

+1

@JingsongLi
JingsongLi merged commit 2d42fb8 into apache:master Jul 23, 2026
11 of 16 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.

2 participants