-
Notifications
You must be signed in to change notification settings - Fork 1.8k
perf: change vectorized equal to in multi column aggregation to use BooleanBufferBuilder instead of &[bool] [WIP]
#19103
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
base: main
Are you sure you want to change the base?
Conversation
|
run benchmark aggregate_vectorized |
|
🤖 Hi @rluvaton, thanks for the request (#19103 (comment)).
Please choose one or more of these with |
|
@alamb anyway we can add this as well? |
|
run benchmarks |
|
🤖 |
Could you provide a PR to https://github.com/alamb/datafusion-benchmarking ? |
|
🤖: Benchmark completed Details
|
|
run benchmarks |
|
🤖 |
|
🤖: Benchmark completed Details
|
|
run benchmarks |
|
🤖 |
|
run benchmarks aggregate_query_sql |
|
🤖 Hi @rluvaton, thanks for the request (#19103 (comment)).
Please choose one or more of these with |
|
🤖: Benchmark completed Details
|
|
I think these benchmarks are grouping on bytes/bytes view columns and I need to further optimize there, but the |
|
run benchmarks aggregate_vectorized aggregate_query_sql |
TODO
Which issue does this PR close?
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?