Add release notes for 0.217#12345
Conversation
|
@nezihyigitbasi Thank you so much for the review! I'll pay more attention to phrasing next time! |
|
@jessesleeping @raghavsethi @rschlussel @wenleix @shishunzhong Could you please take a look at @nezihyigitbasi's comments that I have not marked as resolved? Thanks! |
There was a problem hiding this comment.
Does this have perf impact? If so, let's frame it like that.
There was a problem hiding this comment.
Improve the logical plan by removing unnecessary remote exchanges before window functions when the data is already available locally. This reduces cpu usage and unifies remote exchange elimination strategy between window function and aggregation.
There was a problem hiding this comment.
I'm not sure how user will understand "remote exchange elimination strategy", so I'll rephrase the original ones and add indication that this helps reduce cpu usage.
Improve CPU usage by removing unnecessary remote exchanges from the logical plan before window functions if the data is already available locally.
There was a problem hiding this comment.
Improve performance of some queries that use window functions by eliminating redundant shuffles.
There was a problem hiding this comment.
We can remove this, it's not important.
caithagoras
force-pushed
the
s2
branch
4 times, most recently
from
e9c80dc to
2bd6f9f
Compare
nezihyigitbasi
left a comment
nezihyigitbasi
left a comment
There was a problem hiding this comment.
LGTM % minor comments.
11 participants
#12257