Skip to content

Update docs for aggregate repartition test #18650

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Nov 13, 2025
+0 −3
Merged

Update docs for aggregate repartition test #18650

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0c047f6
update docs for test
xbailey Nov 12, 2025
212644d
Merge branch 'main' into xb/update_test_docs
xanderbailey Nov 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions datafusion/sqllogictest/test_files/aggregate_repartition.slt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,6 @@ physical_plan
07)------------DataSourceExec: file_groups={1 group: [[WORKSPACE_ROOT/datafusion/sqllogictest/test_files/scratch/aggregate_repartition/dim.csv]]}, projection=[env], file_type=csv, has_header=true

# Test 2: EXPLAIN query for Parquet table with GROUP BY
# This plan differs from the one above and includes two consecutive repartitions — one round-robin and one hash —
# which seems unnecessary. We may want to align it with the previous plan (push the round robin down or remove the round robin), or, if the input file is small,
# avoid repartitioning altogether. A single partition should suffice for a single-step aggregate as the plan after this.

query TT
EXPLAIN SELECT env, count(*) FROM dim_parquet GROUP BY env;
Expand Down