[doris] branch master updated: [fix](nereids) fix ut,check bound should be called recursively on the plan node (#10530)

[lingmiao]

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

lingmiao 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 0b9f508379 [fix](nereids) fix ut,check bound should be called 
recursively on the plan node (#10530)
0b9f508379 is described below

commit 0b9f5083790de98e90d306e64bee92300151dd55
Author: yinzhijian <373141...@qq.com>
AuthorDate: Wed Jul 6 10:37:05 2022 +0800

    [fix](nereids) fix ut,check bound should be called recursively on the plan 
node (#10530)
    
    fix ut,check bound should be called recursively on the plan node
---
 .../test/java/org/apache/doris/nereids/AnalyzeSSBTest.java | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git 
a/fe/fe-core/src/test/java/org/apache/doris/nereids/AnalyzeSSBTest.java 
b/fe/fe-core/src/test/java/org/apache/doris/nereids/AnalyzeSSBTest.java
index 49f16a959e..fc742659ea 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/nereids/AnalyzeSSBTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/nereids/AnalyzeSSBTest.java
@@ -161,13 +161,6 @@ public class AnalyzeSSBTest extends TestWithFeService {
         if (!checkPlanBound(root))  {
             return false;
         }
-
-        List<Plan> children = root.children();
-        for (Plan child : children) {
-            if (!checkPlanBound((LogicalPlan) child)) {
-                return false;
-            }
-        }
         return true;
     }
 
@@ -179,6 +172,13 @@ public class AnalyzeSSBTest extends TestWithFeService {
             return false;
         }
 
+        List<Plan> children = plan.children();
+        for (Plan child : children) {
+            if (!checkPlanBound((LogicalPlan) child)) {
+                return false;
+            }
+        }
+
         List<Expression> expressions = plan.getOperator().getExpressions();
         return expressions.stream().allMatch(this::checkExpressionBound);
     }


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