[doris] branch branch-1.2-lts updated: [fix](planner)literal expr should do nothing in substituteImpl() method (#23450)

[morningman]

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


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new cd1e7b1996 [fix](planner)literal expr should do nothing in 
substituteImpl() method (#23450)
cd1e7b1996 is described below

commit cd1e7b1996a763fed09f36b3e9db0bbf381c395c
Author: starocean999 <40539150+starocean...@users.noreply.github.com>
AuthorDate: Fri Aug 25 11:08:45 2023 +0800

    [fix](planner)literal expr should do nothing in substituteImpl() method 
(#23450)
    
    substitute a literal expr is pointless and wrong. This pr keep literal expr 
unchanged during substitute process
    
    pick from master #23438
---
 fe/fe-core/src/main/java/org/apache/doris/analysis/LiteralExpr.java | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/analysis/LiteralExpr.java 
b/fe/fe-core/src/main/java/org/apache/doris/analysis/LiteralExpr.java
index 022f1b41d6..c52d77b2ee 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/LiteralExpr.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/LiteralExpr.java
@@ -267,4 +267,10 @@ public abstract class LiteralExpr extends Expr implements 
Comparable<LiteralExpr
     public String toString() {
         return getStringValue();
     }
+
+    @Override
+    protected Expr substituteImpl(ExprSubstitutionMap smap, 
ExprSubstitutionMap disjunctsMap,
+            Analyzer analyzer) {
+        return this;
+    }
 }


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