From 89f3e1f915ec1031e59ae079e7588418af733008 Mon Sep 17 00:00:00 2001 From: englefly Date: Wed, 1 Nov 2023 13:21:55 +0800 Subject: [PATCH] fix --- .../main/java/org/apache/doris/nereids/cost/CostModelV1.java | 2 +- .../doris/nereids/properties/ChildrenPropertiesRegulator.java | 1 - 2 files changed, 1 insertion(+), 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 index 8b2a09ca07b8043..0cea9a5775536c3 100644 --- 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/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java index 4a2ecdd098b11d9..b65ae5e2d0bd088 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildrenPropertiesRegulator.java @@ -121,7 +121,6 @@ public Boolean visitPhysicalHashAggregate(PhysicalHashAggregate // 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;