[VL] Merge Partial and PartialMerge logic in generateMergeCompanionNo…

…de (apache#4883)
bigo-sg · Oct 8, 2024 · 6b987cf · 6b987cf
1 parent 0856952
commit 6b987cf
Showing 1 changed file with 2 additions and 11 deletions.
diff --git a/backends-velox/src/main/scala/io/glutenproject/execution/HashAggregateExecTransformer.scala b/backends-velox/src/main/scala/io/glutenproject/execution/HashAggregateExecTransformer.scala
@@ -217,18 +217,9 @@ abstract class HashAggregateExecTransformer(
 
     def generateMergeCompanionNode(): Unit = {
       aggregateMode match {
-        case Partial =>
-          val partialNode = ExpressionBuilder.makeAggregateFunction(
-            VeloxAggregateFunctionsBuilder.create(args, aggregateFunction, aggregateMode),
-            childrenNodeList,
-            modeKeyWord,
-            VeloxIntermediateData.getIntermediateTypeNode(aggregateFunction)
-          )
-          aggregateNodeList.add(partialNode)
-        case PartialMerge =>
+        case Partial | PartialMerge =>
           val aggFunctionNode = ExpressionBuilder.makeAggregateFunction(
-            VeloxAggregateFunctionsBuilder
-              .create(args, aggregateFunction, aggregateMode),
+            VeloxAggregateFunctionsBuilder.create(args, aggregateFunction, aggregateMode),
             childrenNodeList,
             modeKeyWord,
             VeloxIntermediateData.getIntermediateTypeNode(aggregateFunction)