Skip to content

[spark] Optimize drop multi partitions in batch#7380

Open
Aitozi wants to merge 3 commits intoapache:masterfrom
Aitozi:multi-partitions
Open

[spark] Optimize drop multi partitions in batch#7380
Aitozi wants to merge 3 commits intoapache:masterfrom
Aitozi:multi-partitions

Conversation

@Aitozi
Copy link
Contributor

@Aitozi Aitozi commented Mar 9, 2026

Purpose

Support to drop multi partial & full partitions. Removing the partition expansion and avoid droping the partition one by one. This could be time cost when the matched partition count is large.

Linked issue: close #xxx

Tests

API and Format

Documentation

Generative AI tooling

@Aitozi
Copy link
Contributor Author

Aitozi commented Mar 9, 2026

CC @baiyangtx pls take a look when you are free

@Aitozi Aitozi force-pushed the multi-partitions branch from 3003c86 to 12b3669 Compare March 9, 2026 12:54
Copy link
Contributor

@baiyangtx baiyangtx left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
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.

I just copy your tests to latest master, all passed.

@Aitozi
Copy link
Contributor Author

Aitozi commented Mar 9, 2026

I just copy your tests to latest master, all passed.

yes, it should pass, it just optimize the process method.

@Aitozi
Copy link
Contributor Author

Aitozi commented Mar 9, 2026

I just copy your tests to latest master, all passed.

yes, it should pass, it just optimize the process method.

Could add a more specific check for the snapshots count, let me add it

@JingsongLi JingsongLi changed the title [spark] Supprt drop multi partitions in batch [spark] Optimize drop multi partitions in batch Mar 9, 2026
@JingsongLi
Copy link
Contributor

I just copy your tests to latest master, all passed.

yes, it should pass, it just optimize the process method.

I see, let's modify the title.

@Aitozi Aitozi force-pushed the multi-partitions branch from 12b3669 to 1114ec7 Compare March 9, 2026 15:03
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