fix

LemonLiTree · Oct 18, 2023 · f105abf · f105abf
1 parent fe788ca
commit f105abf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/...e-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/MapAgg.java b/...e-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/MapAgg.java
@@ -56,12 +56,6 @@ private MapAgg(boolean distinct, Expression arg0, Expression arg1) {
         super("map_agg", distinct, arg0, arg1);
     }
 
-    @Override
-    public FunctionSignature computeSignature(FunctionSignature signature) {
-        signature = signature.withReturnType(MapType.of(getArgumentType(0), getArgumentType(1)));
-        return super.computeSignature(signature);
-    }
-
     /**
      * withDistinctAndChildren.
      */

diff --git a/...ain/java/org/apache/doris/nereids/trees/expressions/visitor/AggregateFunctionVisitor.java b/...ain/java/org/apache/doris/nereids/trees/expressions/visitor/AggregateFunctionVisitor.java
@@ -176,6 +176,10 @@ default R visitHllUnionAgg(HllUnionAgg hllUnionAgg, C context) {
     default R visitIntersectCount(IntersectCount intersectCount, C context) {
         return visitAggregateFunction(intersectCount, context);
     }
+
+    default R visitMapAgg(MapAgg mapAgg, C context) {
+        return visitAggregateFunction(mapAgg, context);
+    }
 
     default R visitMax(Max max, C context) {
         return visitNullableAggregateFunction(max, context);
@@ -285,7 +289,4 @@ default R visitJavaUdaf(JavaUdaf javaUdaf, C context) {
         return visitAggregateFunction(javaUdaf, context);
     }
 
-    default R visitMapAgg(MapAgg mapAgg, C context) {
-        return visitAggregateFunction(mapAgg, context);
-    }
 }