This is an automated email from the ASF dual-hosted git repository.

panxiaolei pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new 2d717aab6b [Chore](pick) fix core dump when lateral view above union 
node and have predicate (#17948)
2d717aab6b is described below

commit 2d717aab6be6e74c845b416aa6d3b02c561f1765
Author: Pxl <pxl...@qq.com>
AuthorDate: Mon Mar 20 14:01:39 2023 +0800

    [Chore](pick) fix core dump when lateral view above union node and have 
predicate (#17948)
    
    cherry-pick #17912 to 1.2
---
 .../src/main/java/org/apache/doris/planner/PlanNode.java  | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java 
b/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java
index 67e0f336d0..4543576a0d 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/PlanNode.java
@@ -618,13 +618,6 @@ public abstract class PlanNode extends TreeNode<PlanNode> 
implements PlanStats {
      * Subclasses need to override this.
      */
     public void finalize(Analyzer analyzer) throws UserException {
-        for (PlanNode child : children) {
-            child.finalize(analyzer);
-        }
-        computeNumNodes();
-        if (!analyzer.safeIsEnableJoinReorderBasedCost()) {
-            computeOldCardinality();
-        }
         for (Expr expr : conjuncts) {
             Set<SlotRef> slotRefs = new HashSet<>();
             expr.getSlotRefsBoundByTupleIds(tupleIds, slotRefs);
@@ -635,6 +628,14 @@ public abstract class PlanNode extends TreeNode<PlanNode> 
implements PlanStats {
                 analyzer.getTupleDesc(tupleId).computeMemLayout();
             }
         }
+
+        for (PlanNode child : children) {
+            child.finalize(analyzer);
+        }
+        computeNumNodes();
+        if (!analyzer.safeIsEnableJoinReorderBasedCost()) {
+            computeOldCardinality();
+        }
     }
 
     protected void computeNumNodes() {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to