perf: do not build parquet pruning predicates if no page index#21556
perf: do not build parquet pruning predicates if no page index#21556alamb wants to merge 3 commits into
Conversation
|
run benchmark clickbench_partitioned |
|
🤖 Benchmark running (GKE) | trigger CPU Details (lscpu)Comparing alamb/less_page_predicates (3017761) to ec00112 (merge-base) diff using: clickbench_partitioned File an issue against this benchmark runner |
|
🤖 Benchmark completed (GKE) | trigger Instance: CPU Details (lscpu)Details
Resource Usageclickbench_partitioned — base (merge-base)
clickbench_partitioned — branch
File an issue against this benchmark runner |
|
run benchmark clickbench_partitioned |
|
🤖 Benchmark running (GKE) | trigger CPU Details (lscpu)Comparing alamb/less_page_predicates (56c95dc) to ec00112 (merge-base) diff using: clickbench_partitioned File an issue against this benchmark runner |
|
🤖 Benchmark completed (GKE) | trigger Instance: CPU Details (lscpu)Details
Resource Usageclickbench_partitioned — base (merge-base)
clickbench_partitioned — branch
File an issue against this benchmark runner |
|
Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days. |
Which issue does this PR close?
Rationale for this change
@Dandandan suggested #21480 (comment)
Which is a good idea to further optimize low latency queries
This also applies to the clickbench_partitioned dataset which has no page indexes
What changes are included in this PR?
Are these changes tested?
Yes
Are there any user-facing changes?
Faster performance (though I am not sure we will be able to measure it)