imay commented on a change in pull request #2952: Fix some function DATE type 
priority
URL: https://github.com/apache/incubator-doris/pull/2952#discussion_r381830172
 
 

 ##########
 File path: fe/src/main/java/org/apache/doris/planner/SingleNodePlanner.java
 ##########
 @@ -802,7 +802,6 @@ private PlanNode createConstantSelectPlan(SelectStmt 
selectStmt, Analyzer analyz
         for (int i = 0; i < resultExprs.size(); ++i) {
             SlotRef slotRef = new SlotRef(tupleDesc.getSlots().get(i));
             resultExprs.set(i, slotRef);
-            selectStmt.getBaseTblResultExprs().set(i, slotRef);
 
 Review comment:
   Is it OK when executing `insert into` statement?
   I remember it seems to be introduced here in support of `insert into 
values()`. You can try if this patch works properly.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to