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

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


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new ffb857b827 [fix](planner) should not change SlotRef to NullLiteral 
(#22635)
ffb857b827 is described below

commit ffb857b827fc07ddda163afb097e36c878d71832
Author: Jerry Hu <[email protected]>
AuthorDate: Sun Aug 6 12:13:33 2023 +0800

    [fix](planner) should not change SlotRef to NullLiteral (#22635)
    
    When executing the NullLiteral expression, a new column will be created, 
whereas executing the SlotRef expression does not create a new column. This 
difference can result in one unexpected column in the block.
---
 fe/fe-core/src/main/java/org/apache/doris/analysis/Expr.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/Expr.java 
b/fe/fe-core/src/main/java/org/apache/doris/analysis/Expr.java
index abed221866..8615d3d803 100755
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/Expr.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/Expr.java
@@ -934,7 +934,7 @@ public abstract class Expr extends TreeNode<Expr> 
implements ParseNode, Cloneabl
             // being cast to a non-NULL type, the type doesn't matter and we 
can cast it
             // arbitrarily.
             Preconditions.checkState(this instanceof NullLiteral || this 
instanceof SlotRef);
-            return NullLiteral.create(ScalarType.BOOLEAN).treeToThrift();
+            type = ScalarType.BOOLEAN;
         }
         TExpr result = new TExpr();
         treeToThriftHelper(result);


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

Reply via email to