-
Notifications
You must be signed in to change notification settings - Fork 590
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
perf: nexmark q15 #7350
Comments
7 tasks
mergify bot
pushed a commit
that referenced
this issue
Feb 24, 2023
Previously in #7797, distinct agg support is added (without cache) but not enabled. This PR enables it by disable 2-phase rewrite rule for streaming distinct agg calls, and also adds an LRU cache in the deduplicater. This will close #7682, and possibly resolve or at least mitigate the performance issue in #7350 and #7271. Approved-By: st1page
Now 3/2: RW:
|
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
query:
plan:
1/13:
The same as #7271 , requires changing to shuffle aggregation instead of two-phase aggregation.
Check again after it is done.
The text was updated successfully, but these errors were encountered: