Skip to content

Commit 5a6db9f

Browse files
authored
chore: improve RepartitionExec display tree (#15606)
* fix RepartitionExec tree displayformat * pass sqllogictests
1 parent b116ca3 commit 5a6db9f

File tree

2 files changed

+39
-39
lines changed

2 files changed

+39
-39
lines changed

datafusion/physical-plan/src/repartition/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ impl DisplayAs for RepartitionExec {
510510
writeln!(f, "partitioning_scheme={}", self.partitioning(),)?;
511511
writeln!(
512512
f,
513-
"output_partition_count={}",
513+
"input_partition_count={}",
514514
self.input.output_partitioning().partition_count()
515515
)?;
516516
if self.preserve_order {

datafusion/sqllogictest/test_files/explain_tree.slt

Lines changed: 38 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ physical_plan
180180
13)┌─────────────┴─────────────┐
181181
14)│ RepartitionExec │
182182
15)│ -------------------- │
183-
16)│ output_partition_count: │
183+
16)│ input_partition_count: │
184184
17)│ 1 │
185185
18)│ │
186186
19)│ partitioning_scheme: │
@@ -218,7 +218,7 @@ physical_plan
218218
18)┌─────────────┴─────────────┐
219219
19)│ RepartitionExec │
220220
20)│ -------------------- │
221-
21)│ output_partition_count: │
221+
21)│ input_partition_count: │
222222
22)│ 4 │
223223
23)│ │
224224
24)│ partitioning_scheme: │
@@ -236,7 +236,7 @@ physical_plan
236236
36)┌─────────────┴─────────────┐
237237
37)│ RepartitionExec │
238238
38)│ -------------------- │
239-
39)│ output_partition_count: │
239+
39)│ input_partition_count: │
240240
40)│ 1 │
241241
41)│ │
242242
42)│ partitioning_scheme: │
@@ -311,7 +311,7 @@ physical_plan
311311
19)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
312312
20)│ RepartitionExec ││ RepartitionExec │
313313
21)│ -------------------- ││ -------------------- │
314-
22)│ output_partition_count: ││ output_partition_count: │
314+
22)│ input_partition_count: ││ input_partition_count: │
315315
23)│ 4 ││ 4 │
316316
24)│ ││ │
317317
25)│ partitioning_scheme: ││ partitioning_scheme: │
@@ -320,7 +320,7 @@ physical_plan
320320
28)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
321321
29)│ RepartitionExec ││ RepartitionExec │
322322
30)│ -------------------- ││ -------------------- │
323-
31)│ output_partition_count: ││ output_partition_count: │
323+
31)│ input_partition_count: ││ input_partition_count: │
324324
32)│ 1 ││ 1 │
325325
33)│ ││ │
326326
34)│ partitioning_scheme: ││ partitioning_scheme: │
@@ -386,7 +386,7 @@ physical_plan
386386
40)-----------------------------┌─────────────┴─────────────┐┌─────────────┴─────────────┐
387387
41)-----------------------------│ RepartitionExec ││ RepartitionExec │
388388
42)-----------------------------│ -------------------- ││ -------------------- │
389-
43)-----------------------------│ output_partition_count: ││ output_partition_count: │
389+
43)-----------------------------│ input_partition_count: ││ input_partition_count: │
390390
44)-----------------------------│ 4 ││ 4 │
391391
45)-----------------------------│ ││ │
392392
46)-----------------------------│ partitioning_scheme: ││ partitioning_scheme: │
@@ -395,7 +395,7 @@ physical_plan
395395
49)-----------------------------┌─────────────┴─────────────┐┌─────────────┴─────────────┐
396396
50)-----------------------------│ RepartitionExec ││ RepartitionExec │
397397
51)-----------------------------│ -------------------- ││ -------------------- │
398-
52)-----------------------------│ output_partition_count: ││ output_partition_count: │
398+
52)-----------------------------│ input_partition_count: ││ input_partition_count: │
399399
53)-----------------------------│ 1 ││ 1 │
400400
54)-----------------------------│ ││ │
401401
55)-----------------------------│ partitioning_scheme: ││ partitioning_scheme: │
@@ -434,7 +434,7 @@ physical_plan
434434
17)┌─────────────┴─────────────┐
435435
18)│ RepartitionExec │
436436
19)│ -------------------- │
437-
20)│ output_partition_count: │
437+
20)│ input_partition_count: │
438438
21)│ 1 │
439439
22)│ │
440440
23)│ partitioning_scheme: │
@@ -496,7 +496,7 @@ physical_plan
496496
41)┌─────────────┴─────────────┐
497497
42)│ RepartitionExec │
498498
43)│ -------------------- │
499-
44)│ output_partition_count: │
499+
44)│ input_partition_count: │
500500
45)│ 1 │
501501
46)│ │
502502
47)│ partitioning_scheme: │
@@ -530,7 +530,7 @@ physical_plan
530530
13)┌─────────────┴─────────────┐
531531
14)│ RepartitionExec │
532532
15)│ -------------------- │
533-
16)│ output_partition_count: │
533+
16)│ input_partition_count: │
534534
17)│ 1 │
535535
18)│ │
536536
19)│ partitioning_scheme: │
@@ -566,7 +566,7 @@ physical_plan
566566
15)┌─────────────┴─────────────┐
567567
16)│ RepartitionExec │
568568
17)│ -------------------- │
569-
18)│ output_partition_count: │
569+
18)│ input_partition_count: │
570570
19)│ 1 │
571571
20)│ │
572572
21)│ partitioning_scheme: │
@@ -599,7 +599,7 @@ physical_plan
599599
13)┌─────────────┴─────────────┐
600600
14)│ RepartitionExec │
601601
15)│ -------------------- │
602-
16)│ output_partition_count: │
602+
16)│ input_partition_count: │
603603
17)│ 1 │
604604
18)│ │
605605
19)│ partitioning_scheme: │
@@ -633,7 +633,7 @@ physical_plan
633633
13)┌─────────────┴─────────────┐
634634
14)│ RepartitionExec │
635635
15)│ -------------------- │
636-
16)│ output_partition_count: │
636+
16)│ input_partition_count: │
637637
17)│ 1 │
638638
18)│ │
639639
19)│ partitioning_scheme: │
@@ -694,7 +694,7 @@ physical_plan
694694
13)┌─────────────┴─────────────┐
695695
14)│ RepartitionExec │
696696
15)│ -------------------- │
697-
16)│ output_partition_count: │
697+
16)│ input_partition_count: │
698698
17)│ 1 │
699699
18)│ │
700700
19)│ partitioning_scheme: │
@@ -727,7 +727,7 @@ physical_plan
727727
13)┌─────────────┴─────────────┐
728728
14)│ RepartitionExec │
729729
15)│ -------------------- │
730-
16)│ output_partition_count: │
730+
16)│ input_partition_count: │
731731
17)│ 1 │
732732
18)│ │
733733
19)│ partitioning_scheme: │
@@ -922,7 +922,7 @@ physical_plan
922922
13)┌─────────────┴─────────────┐
923923
14)│ RepartitionExec │
924924
15)│ -------------------- │
925-
16)│ output_partition_count: │
925+
16)│ input_partition_count: │
926926
17)│ 1 │
927927
18)│ │
928928
19)│ partitioning_scheme: │
@@ -1031,7 +1031,7 @@ physical_plan
10311031
13)┌─────────────┴─────────────┐
10321032
14)│ RepartitionExec │
10331033
15)│ -------------------- │
1034-
16)│ output_partition_count: │
1034+
16)│ input_partition_count: │
10351035
17)│ 1 │
10361036
18)│ │
10371037
19)│ partitioning_scheme: │
@@ -1089,7 +1089,7 @@ physical_plan
10891089
12)┌─────────────┴─────────────┐
10901090
13)│ RepartitionExec │
10911091
14)│ -------------------- │
1092-
15)│ output_partition_count: │
1092+
15)│ input_partition_count: │
10931093
16)│ 1 │
10941094
17)│ │
10951095
18)│ partitioning_scheme: │
@@ -1123,7 +1123,7 @@ physical_plan
11231123
13)┌─────────────┴─────────────┐
11241124
14)│ RepartitionExec │
11251125
15)│ -------------------- │
1126-
16)│ output_partition_count: │
1126+
16)│ input_partition_count: │
11271127
17)│ 1 │
11281128
18)│ │
11291129
19)│ partitioning_scheme: │
@@ -1209,7 +1209,7 @@ physical_plan
12091209
22)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
12101210
23)│ RepartitionExec ││ RepartitionExec │
12111211
24)│ -------------------- ││ -------------------- │
1212-
25)│ output_partition_count: ││ output_partition_count: │
1212+
25)│ input_partition_count: ││ input_partition_count: │
12131213
26)│ 4 ││ 4 │
12141214
27)│ ││ │
12151215
28)│ partitioning_scheme: ││ partitioning_scheme: │
@@ -1220,7 +1220,7 @@ physical_plan
12201220
33)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
12211221
34)│ ProjectionExec ││ RepartitionExec │
12221222
35)│ -------------------- ││ -------------------- │
1223-
36)│ CAST(table1.string_col AS ││ output_partition_count: │
1223+
36)│ CAST(table1.string_col AS ││ input_partition_count: │
12241224
37)│ Utf8View): ││ 1 │
12251225
38)│ CAST(string_col AS ││ │
12261226
39)│ Utf8View) ││ partitioning_scheme: │
@@ -1237,7 +1237,7 @@ physical_plan
12371237
50)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
12381238
51)│ RepartitionExec ││ DataSourceExec │
12391239
52)│ -------------------- ││ -------------------- │
1240-
53)│ output_partition_count: ││ files: 1 │
1240+
53)│ input_partition_count: ││ files: 1 │
12411241
54)│ 1 ││ format: parquet │
12421242
55)│ ││ │
12431243
56)│ partitioning_scheme: ││ │
@@ -1281,7 +1281,7 @@ physical_plan
12811281
24)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
12821282
25)│ RepartitionExec ││ RepartitionExec │
12831283
26)│ -------------------- ││ -------------------- │
1284-
27)│ output_partition_count: ││ output_partition_count: │
1284+
27)│ input_partition_count: ││ input_partition_count: │
12851285
28)│ 4 ││ 4 │
12861286
29)│ ││ │
12871287
30)│ partitioning_scheme: ││ partitioning_scheme: │
@@ -1292,7 +1292,7 @@ physical_plan
12921292
35)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
12931293
36)│ ProjectionExec ││ RepartitionExec │
12941294
37)│ -------------------- ││ -------------------- │
1295-
38)│ CAST(table1.string_col AS ││ output_partition_count: │
1295+
38)│ CAST(table1.string_col AS ││ input_partition_count: │
12961296
39)│ Utf8View): ││ 1 │
12971297
40)│ CAST(string_col AS ││ │
12981298
41)│ Utf8View) ││ partitioning_scheme: │
@@ -1309,7 +1309,7 @@ physical_plan
13091309
52)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
13101310
53)│ RepartitionExec ││ DataSourceExec │
13111311
54)│ -------------------- ││ -------------------- │
1312-
55)│ output_partition_count: ││ files: 1 │
1312+
55)│ input_partition_count: ││ files: 1 │
13131313
56)│ 1 ││ format: parquet │
13141314
57)│ ││ │
13151315
58)│ partitioning_scheme: ││ │
@@ -1356,7 +1356,7 @@ physical_plan
13561356
27)-----------------------------┌─────────────┴─────────────┐
13571357
28)-----------------------------│ RepartitionExec │
13581358
29)-----------------------------│ -------------------- │
1359-
30)-----------------------------│ output_partition_count: │
1359+
30)-----------------------------│ input_partition_count: │
13601360
31)-----------------------------│ 1 │
13611361
32)-----------------------------│ │
13621362
33)-----------------------------│ partitioning_scheme: │
@@ -1380,7 +1380,7 @@ physical_plan
13801380
04)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
13811381
05)│ DataSourceExec ││ RepartitionExec │
13821382
06)│ -------------------- ││ -------------------- │
1383-
07)│ files: 1 ││ output_partition_count: │
1383+
07)│ files: 1 ││ input_partition_count: │
13841384
08)│ format: csv ││ 1 │
13851385
09)│ ││ │
13861386
10)│ ││ partitioning_scheme: │
@@ -1505,7 +1505,7 @@ physical_plan
15051505
33)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
15061506
34)│ RepartitionExec ││ RepartitionExec │
15071507
35)│ -------------------- ││ -------------------- │
1508-
36)│ output_partition_count: ││ output_partition_count: │
1508+
36)│ input_partition_count: ││ input_partition_count: │
15091509
37)│ 4 ││ 4 │
15101510
38)│ ││ │
15111511
39)│ partitioning_scheme: ││ partitioning_scheme: │
@@ -1514,7 +1514,7 @@ physical_plan
15141514
42)┌─────────────┴─────────────┐┌─────────────┴─────────────┐
15151515
43)│ RepartitionExec ││ RepartitionExec │
15161516
44)│ -------------------- ││ -------------------- │
1517-
45)│ output_partition_count: ││ output_partition_count: │
1517+
45)│ input_partition_count: ││ input_partition_count: │
15181518
46)│ 1 ││ 1 │
15191519
47)│ ││ │
15201520
48)│ partitioning_scheme: ││ partitioning_scheme: │
@@ -1606,7 +1606,7 @@ physical_plan
16061606
18)┌─────────────┴─────────────┐
16071607
19)│ RepartitionExec │
16081608
20)│ -------------------- │
1609-
21)│ output_partition_count: │
1609+
21)│ input_partition_count: │
16101610
22)│ 1 │
16111611
23)│ │
16121612
24)│ partitioning_scheme: │
@@ -1648,7 +1648,7 @@ physical_plan
16481648
19)┌─────────────┴─────────────┐
16491649
20)│ RepartitionExec │
16501650
21)│ -------------------- │
1651-
22)│ output_partition_count: │
1651+
22)│ input_partition_count: │
16521652
23)│ 1 │
16531653
24)│ │
16541654
25)│ partitioning_scheme: │
@@ -1689,7 +1689,7 @@ physical_plan
16891689
19)┌─────────────┴─────────────┐
16901690
20)│ RepartitionExec │
16911691
21)│ -------------------- │
1692-
22)│ output_partition_count: │
1692+
22)│ input_partition_count: │
16931693
23)│ 1 │
16941694
24)│ │
16951695
25)│ partitioning_scheme: │
@@ -1728,7 +1728,7 @@ physical_plan
17281728
17)┌─────────────┴─────────────┐
17291729
18)│ RepartitionExec │
17301730
19)│ -------------------- │
1731-
20)│ output_partition_count: │
1731+
20)│ input_partition_count: │
17321732
21)│ 1 │
17331733
22)│ │
17341734
23)│ partitioning_scheme: │
@@ -1771,7 +1771,7 @@ physical_plan
17711771
20)┌─────────────┴─────────────┐
17721772
21)│ RepartitionExec │
17731773
22)│ -------------------- │
1774-
23)│ output_partition_count: │
1774+
23)│ input_partition_count: │
17751775
24)│ 1 │
17761776
25)│ │
17771777
26)│ partitioning_scheme: │
@@ -1815,7 +1815,7 @@ physical_plan
18151815
19)┌─────────────┴─────────────┐
18161816
20)│ RepartitionExec │
18171817
21)│ -------------------- │
1818-
22)│ output_partition_count: │
1818+
22)│ input_partition_count: │
18191819
23)│ 1 │
18201820
24)│ │
18211821
25)│ partitioning_scheme: │
@@ -1869,7 +1869,7 @@ physical_plan
18691869
25)-----------------------------┌─────────────┴─────────────┐
18701870
26)-----------------------------│ RepartitionExec │
18711871
27)-----------------------------│ -------------------- │
1872-
28)-----------------------------│ output_partition_count: │
1872+
28)-----------------------------│ input_partition_count: │
18731873
29)-----------------------------│ 1 │
18741874
30)-----------------------------│ │
18751875
31)-----------------------------│ partitioning_scheme: │
@@ -1983,7 +1983,7 @@ physical_plan
19831983
22)┌─────────────┴─────────────┐
19841984
23)│ RepartitionExec │
19851985
24)│ -------------------- │
1986-
25)│ output_partition_count: │
1986+
25)│ input_partition_count: │
19871987
26)│ 1 │
19881988
27)│ │
19891989
28)│ partitioning_scheme: │
@@ -2062,7 +2062,7 @@ physical_plan
20622062
19)┌─────────────┴─────────────┐
20632063
20)│ RepartitionExec │
20642064
21)│ -------------------- │
2065-
22)│ output_partition_count: │
2065+
22)│ input_partition_count: │
20662066
23)│ 1 │
20672067
24)│ │
20682068
25)│ partitioning_scheme: │

0 commit comments

Comments
 (0)