You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2023/06/16 06:54:28 UTC
[doris] branch master updated: [fix](nereids) only setHasColocatePlanNode when currentFragment's data partition is not RANDOM (#20885)
This is an automated email from the ASF dual-hosted git repository.
morrysnow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new ccfd6f1d23 [fix](nereids) only setHasColocatePlanNode when currentFragment's data partition is not RANDOM (#20885)
ccfd6f1d23 is described below
commit ccfd6f1d23706f7c026e6fb728169f3b4020cf3b
Author: starocean999 <40...@users.noreply.github.com>
AuthorDate: Fri Jun 16 14:54:22 2023 +0800
[fix](nereids) only setHasColocatePlanNode when currentFragment's data partition is not RANDOM (#20885)
---
.../apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java
index 2a961d70e3..8593b750ba 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/PhysicalPlanTranslator.java
@@ -455,7 +455,8 @@ public class PhysicalPlanTranslator extends DefaultPlanVisitor<PlanFragment, Pla
// TODO: nereids forbid all parallel scan under aggregate temporary, because nereids could generate
// so complex aggregate plan than legacy planner, and should add forbid parallel scan hint when
// generate physical aggregate plan.
- if (leftMostNode instanceof OlapScanNode) {
+ if (leftMostNode instanceof OlapScanNode
+ && currentFragment.getDataPartition().getType() != TPartitionType.RANDOM) {
currentFragment.setHasColocatePlanNode(true);
}
setPlanRoot(currentFragment, aggregationNode, aggregate);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org