You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by ja...@apache.org on 2022/10/13 05:49:29 UTC
[doris] branch master updated: [improve](Nereids): avoid duplicated stats derive. (#13293)
This is an automated email from the ASF dual-hosted git repository.
jakevin 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 4248c6f37c [improve](Nereids): avoid duplicated stats derive. (#13293)
4248c6f37c is described below
commit 4248c6f37cb4d3e4f5fcfab29c539aac77849f8c
Author: jakevin <ja...@gmail.com>
AuthorDate: Thu Oct 13 13:49:21 2022 +0800
[improve](Nereids): avoid duplicated stats derive. (#13293)
---
.../java/org/apache/doris/nereids/jobs/cascades/DeriveStatsJob.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/cascades/DeriveStatsJob.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/cascades/DeriveStatsJob.java
index 8797327311..618382438a 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/cascades/DeriveStatsJob.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/cascades/DeriveStatsJob.java
@@ -60,8 +60,9 @@ public class DeriveStatsJob extends Job {
deriveChildren = true;
pushJob(new DeriveStatsJob(this));
for (Group child : groupExpression.children()) {
- if (!child.getLogicalExpressions().isEmpty()) {
- pushJob(new DeriveStatsJob(child.getLogicalExpressions().get(0), context));
+ GroupExpression childGroupExpr = child.getLogicalExpressions().get(0);
+ if (!child.getLogicalExpressions().isEmpty() && !childGroupExpr.isStatDerived()) {
+ pushJob(new DeriveStatsJob(childGroupExpr, context));
}
}
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org