fix

englefly · Nov 1, 2023 · 89f3e1f · 89f3e1f
1 parent 1aa0da6
commit 89f3e1f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/cost/CostModelV1.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/cost/CostModelV1.java
@@ -227,7 +227,7 @@ public Cost visitPhysicalDistribute(
         }
 
         // any
-        // cost of randome shuffle is lower than hash shuffle.
+        // cost of random shuffle is lower than hash shuffle.
         return CostV1.of(
                 0,
                 0,

diff --git a/...e-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java b/...e-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java
@@ -121,7 +121,6 @@ public Boolean visitPhysicalHashAggregate(PhysicalHashAggregate<? extends Plan>
         // no matter x.ndv is high or not, it is not worthwhile to shuffle A and B by x
         // and hence we forbid one phase agg
         if (agg.getAggMode() == AggMode.INPUT_TO_RESULT
-                && requiredProperties.get(0).getDistributionSpec() instanceof DistributionSpecHash
                 && children.get(0).getPlan() instanceof PhysicalUnion
                 && !((PhysicalUnion) children.get(0).getPlan()).isDistinct()) {
             return false;
-Original file line number
+Diff line change
@@ Expand Up / @@ -227,7 +227,7 @@ public Cost visitPhysicalDistribute( @@
             }
             // any
-            // cost of randome shuffle is lower than hash shuffle.
+            // cost of random shuffle is lower than hash shuffle.
             return CostV1.of(
 ,
 ,
@@ Expand Down @@