[doris] branch master updated: [fix](planner)shouldn't force push down conjuncts for union statement (#22079)

[starocean999]

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

starocean999 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 acf4aa2818 [fix](planner)shouldn't force push down conjuncts for union 
statement (#22079)
acf4aa2818 is described below

commit acf4aa281839fe25b440867632532661b9e4791c
Author: starocean999 <40539150+starocean...@users.noreply.github.com>
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 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<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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org