branch-2.1: [fix](planner) FunctionCallExpr clone should be deep copy #…

…44678 (#44711) Cherry-picked from #44678 Co-authored-by: morrySnow <[email protected]>
apache · Nov 29, 2024 · 6f868e4 · 6f868e4
1 parent fc942c4
commit 6f868e4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/FunctionCallExpr.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/FunctionCallExpr.java
@@ -430,7 +430,7 @@ public FunctionCallExpr(FunctionCallExpr e, FunctionParams params) {
 
     protected FunctionCallExpr(FunctionCallExpr other) {
         super(other);
-        fnName = other.fnName;
+        fnName = other.fnName != null ? other.fnName.clone() : null;
         orderByElements = other.orderByElements;
         isAnalyticFnCall = other.isAnalyticFnCall;
         // aggOp = other.aggOp;

diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/FunctionName.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/FunctionName.java
@@ -193,4 +193,9 @@ public static FunctionName read(DataInput in) throws IOException {
     public int hashCode() {
         return 31 * Objects.hashCode(db) + Objects.hashCode(fn);
     }
+
+    @Override
+    public FunctionName clone() {
+        return new FunctionName(db, fn);
+    }
 }