Skip to content

Conversation

@0xPoe
Copy link
Contributor

@0xPoe 0xPoe commented Nov 7, 2025

Which issue does this PR close?

Rationale for this change

What changes are included in this PR?

Implemented partition_statistics for WindowAggExec.

Are these changes tested?

Check the unit tests.

Are there any user-facing changes?

No

@github-actions github-actions bot added core Core DataFusion crate physical-plan Changes to the physical-plan crate labels Nov 7, 2025
@Jefffrey
Copy link
Contributor

@xudong963 perhaps you can take a look at this?

@xudong963
Copy link
Member

Yeah, sorry, I totally missed the PR

Copy link
Member

@xudong963 xudong963 left a comment

Choose a reason for hiding this comment

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

@0xPoe LGTM, thank you

@0xPoe
Copy link
Contributor Author

0xPoe commented Dec 30, 2025

I will resolve the conflict later.

@0xPoe 0xPoe force-pushed the poe-patch-window-func branch 3 times, most recently from 482d7e6 to 6b59d4a Compare December 30, 2025 20:53
@0xPoe 0xPoe requested a review from xudong963 December 30, 2025 20:53
@0xPoe 0xPoe force-pushed the poe-patch-window-func branch 2 times, most recently from 3ecc723 to 6b59d4a Compare December 30, 2025 20:54
@0xPoe 0xPoe force-pushed the poe-patch-window-func branch from 6b59d4a to 11b7476 Compare December 30, 2025 20:55
@0xPoe
Copy link
Contributor Author

0xPoe commented Dec 30, 2025

Rebased onto the main branch.

@xudong963 xudong963 added this pull request to the merge queue Dec 31, 2025
Merged via the queue into apache:main with commit 79f67b8 Dec 31, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core DataFusion crate physical-plan Changes to the physical-plan crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants