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

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git

commit f5d62012f6a42f03937b9aed58b7e984d86ad67c
Author: morrySnow <[email protected]>
AuthorDate: Wed Mar 22 10:53:15 2023 +0800

    [fix](planner) should always execute projection plan (#17885)
    
    1. should always execute projection plan, whatever the statement it is.
    2. should always execute projection plan, since we only have vectorized 
engine now
---
 .../src/main/java/org/apache/doris/planner/OriginalPlanner.java   | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/planner/OriginalPlanner.java 
b/fe/fe-core/src/main/java/org/apache/doris/planner/OriginalPlanner.java
index 809bb645b3..8a11c012b1 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/OriginalPlanner.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/OriginalPlanner.java
@@ -151,12 +151,8 @@ public class OriginalPlanner extends Planner {
             singleNodePlan.convertToVectoriezd();
         }
 
-        if (analyzer.getContext() != null
-                && 
analyzer.getContext().getSessionVariable().isEnableProjection()
-                && statement instanceof QueryStmt) {
-            ProjectPlanner projectPlanner = new ProjectPlanner(analyzer);
-            projectPlanner.projectSingleNodePlan(queryStmt.getResultExprs(), 
singleNodePlan);
-        }
+        ProjectPlanner projectPlanner = new ProjectPlanner(analyzer);
+        projectPlanner.projectSingleNodePlan(queryStmt.getResultExprs(), 
singleNodePlan);
 
         if (statement instanceof InsertStmt) {
             InsertStmt insertStmt = (InsertStmt) statement;


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to