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 <40539150+starocean...@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-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
