From acf4aa281839fe25b440867632532661b9e4791c Mon Sep 17 00:00:00 2001 From: starocean999 <40539150+starocean999@users.noreply.github.com> Date: Fri, 21 Jul 2023 21:12:56 +0800 Subject: [PATCH] [fix](planner)shouldn't force push down conjuncts for union statement (#22079) * [fix](planner)shouldn't force push down conjuncts for union statement --- .../org/apache/doris/planner/SingleNodePlanner.java | 12 ------------ .../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 0e0fceb4c4..6365f5ae97 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 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 {