This is an automated email from the ASF dual-hosted git repository. kxiao pushed a commit to branch branch-2.0 in repository https://gitbox.apache.org/repos/asf/doris.git
commit edf236f574920b701de57480c6f04376fa5c3792 Author: starocean999 <[email protected]> AuthorDate: Fri Jul 21 21:12:56 2023 +0800 [fix](planner)shouldn't force push down conjuncts for union statement (#22079) * [fix](planner)shouldn't force push down conjuncts for union statement --- .../java/org/apache/doris/planner/SingleNodePlanner.java | 12 ------------ .../correctness_p0/test_push_conjuncts_inlineview.groovy | 2 +- 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java b/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java index 488b09b6cc..e7f130ee36 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java +++ b/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java @@ -1823,18 +1823,6 @@ public class SingleNodePlanner { e.setIsOnClauseConjunct(false); } inlineViewRef.getAnalyzer().registerConjuncts(viewPredicates, inlineViewRef.getAllTupleIds()); - QueryStmt queryStmt = inlineViewRef.getQueryStmt(); - if (queryStmt instanceof SetOperationStmt) { - // registerConjuncts for every set operand - SetOperationStmt setOperationStmt = (SetOperationStmt) queryStmt; - for (SetOperationStmt.SetOperand setOperand : setOperationStmt.getOperands()) { - setOperand.getAnalyzer().registerConjuncts( - Expr.substituteList(viewPredicates, setOperand.getSmap(), - setOperand.getAnalyzer(), false), - inlineViewRef.getAllTupleIds()); - } - } - // mark (fully resolve) slots referenced by remaining unassigned conjuncts as // materialized List<Expr> substUnassigned = Expr.substituteList(unassignedConjuncts, diff --git a/regression-test/suites/correctness_p0/test_push_conjuncts_inlineview.groovy b/regression-test/suites/correctness_p0/test_push_conjuncts_inlineview.groovy index 462dfed26d..7cadefc83a 100644 --- a/regression-test/suites/correctness_p0/test_push_conjuncts_inlineview.groovy +++ b/regression-test/suites/correctness_p0/test_push_conjuncts_inlineview.groovy @@ -57,7 +57,7 @@ suite("test_push_conjuncts_inlineview") { )a where a.px = 1;""") - contains "4:VSELECT" + contains "5:VSELECT" } explain { --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
