From 0b9f5083790de98e90d306e64bee92300151dd55 Mon Sep 17 00:00:00 2001 From: yinzhijian <373141588@qq.com> Date: Wed, 6 Jul 2022 10:37:05 +0800 Subject: [PATCH] [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 --- .../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 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 children = plan.children(); + for (Plan child : children) { + if (!checkPlanBound((LogicalPlan) child)) { + return false; + } + } + List expressions = plan.getOperator().getExpressions(); return expressions.stream().allMatch(this::checkExpressionBound); }