From a4b151e469d8d3f6672f8e6741c83434a63244aa Mon Sep 17 00:00:00 2001
From: morrySnow <101034200+morrySnow@users.noreply.github.com>
Date: Wed, 22 Mar 2023 10:53:15 +0800
Subject: [PATCH] [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
---
 .../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 25c6c62bc6..21bd1366fe 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
@@ -165,12 +165,8 @@ public class OriginalPlanner extends Planner {
             singleNodePlan.convertToVectorized();
         }
 
-        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;