Skip to content

[native] Add remaining from spill header to http shuffle API#25271

Merged
tanjialiang merged 1 commit intoprestodb:masterfrom
tanjialiang:export-D76286914
Jun 9, 2025
Merged

[native] Add remaining from spill header to http shuffle API#25271
tanjialiang merged 1 commit intoprestodb:masterfrom
tanjialiang:export-D76286914

Conversation

@tanjialiang
Copy link
Contributor

Summary: To support PartitionedOutput spilling, we need an additional flag telling the receiving end if the remaining buffer will be from spill, for the receiving end to make proper load decisions.

Differential Revision: D76286914

Summary: To support PartitionedOutput spilling, we need an additional flag telling the receiving end if the remaining buffer will be from spill, for the receiving end to make proper load decisions.

Differential Revision: D76286914
@tanjialiang tanjialiang requested a review from a team as a code owner June 9, 2025 19:25
@prestodb-ci prestodb-ci added the from:Meta PR from Meta label Jun 9, 2025
@linux-foundation-easycla
Copy link

CLA Missing ID CLA Not Signed

@facebook-github-bot
Copy link
Collaborator

This pull request was exported from Phabricator. Differential Revision: D76286914

Copy link
Contributor

@spershin spershin left a comment

Choose a reason for hiding this comment

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

Looks good.

@tanjialiang tanjialiang merged commit 22e4de1 into prestodb:master Jun 9, 2025
107 of 110 checks passed
@prestodb-ci prestodb-ci mentioned this pull request Jul 28, 2025
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants