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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
