docs: refresh benchmarking.md with SF1000 results after #2315 (AQE default-on) - #2336
Open
andygrove wants to merge 1 commit into
Open
docs: refresh benchmarking.md with SF1000 results after #2315 (AQE default-on)#2336andygrove wants to merge 1 commit into
andygrove wants to merge 1 commit into
Conversation
…pache#2315) Same reference cluster and configuration as the previous refresh (696ca29). Ballista rebuilt at the merge commit of apache#2315 (AQE default-on, max_partitions_per_task=0 default). Highlights: - Q11 now completes (was failing on the pre-apache#2315 build with the client-side 16 MiB gRPC OutOfRange). AQE plan rewrites shrink Q11's encoded plan below the raised 128 MiB ceiling. - Q10 regressed (was 52.99s, now FAIL) — a SortPreservingMergeExec exhausts its per-task fair memory pool under the new AQE plan. Tracked in apache#2321. - Q21, Q22 still fail; the observed failures were connection-level (h2 body / tcp connect) after Q20, consistent with the scheduler becoming unresponsive at end of suite. - Total on the comparable 18-query subset (Q1-Q9, Q11-Q20): Ballista 617.51s vs Spark 3.4 664.17s. Also documents that ballista.scheduler.max_partitions_per_task now defaults to 0 (unbounded) since apache#2315, previously 1.
Dandandan
approved these changes
Aug 17, 2026
Contributor
|
Would it be possible to compare ballista and spark plans? it might give us some clue where to look, is it planning or physical operators or infrastructure or all previous the problem |
Contributor
I looked at SF=10 plans yesterday, many plans looks pretty good but there are some categories:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docs/source/contributors-guide/benchmarking.mdwith a new SF1000 run atf21c958f(the merge commit of fix: repair AQE plan rewrites, remove a redundant shuffle, and enable adaptive query planning by default #2315). Same reference cluster (32× r6i.24xlarge, 8 vCPU / 64 GiB per executor, gp3 shuffle PVCs) and configuration as the previous refresh at696ca29b.mainafter fix: repair AQE plan rewrites, remove a redundant shuffle, and enable adaptive query planning by default #2315 was merged; DataFusion is now on55.0.0-rc3(Arrow59.2.0).ballista.scheduler.max_partitions_per_tasknow defaults to0(unbounded) since fix: repair AQE plan rewrites, remove a redundant shuffle, and enable adaptive query planning by default #2315, and annotateballista.planner.adaptive.enabledas the compiled-in default since fix: repair AQE plan rewrites, remove a redundant shuffle, and enable adaptive query planning by default #2315.Result changes vs previous refresh
Q11 now completes (was failing on the pre-#2315 build with the client-side 16 MiB gRPC
OutOfRange). The AQE plan rewrites in #2315 shrink Q11's encoded plan below the raised 128 MiB ceiling.Q10 regressed (was 52.99s, now FAIL) — a
SortPreservingMergeExecexhausts its per-task fair memory pool under the new AQE plan. Tracked in #2321.Q21, Q22 still fail; observed failures were connection-level (h2 body read / TCP connect refused) after Q20, consistent with the scheduler becoming unresponsive at end of suite.
Total on the comparable 18-query subset (Q1–Q9, Q11–Q20): Ballista 617.51s vs Spark 3.4 664.17s.
Test plan
docs/build.shrenders benchmarking.md without warnings