unity agg output

apache · Nov 5, 2024 · 3266361 · 3266361
1 parent 48d312a
commit 3266361
Showing 1 changed file with 23 additions and 3 deletions.
diff --git a/...ickhouse/src/main/scala/org/apache/gluten/backendsapi/clickhouse/CHSparkPlanExecApi.scala b/...ickhouse/src/main/scala/org/apache/gluten/backendsapi/clickhouse/CHSparkPlanExecApi.scala
@@ -158,16 +158,36 @@ class CHSparkPlanExecApi extends SparkPlanExecApi with Logging {
       aggregateAttributes: Seq[Attribute],
       initialInputBufferOffset: Int,
       resultExpressions: Seq[NamedExpression],
-      child: SparkPlan): HashAggregateExecBaseTransformer =
-    CHHashAggregateExecTransformer(
+      child: SparkPlan): HashAggregateExecBaseTransformer = {
+    logError(s"xxx aggregateExpressions:$aggregateExpressions")
+    logError(s"xxx aggregateAttributes:$aggregateAttributes")
+    logError(s"xxx resultExpressions:$resultExpressions")
+    logError(s"xxx agg expr to result: ${aggregateExpressions.map(_.resultAttribute)}")
+    logError(
+      s"xxx agg:" +
+        s"${aggregateExpressions.map(e => e.aggregateFunction.aggBufferAttributes.length)}")
+    aggregateExpressions.foreach {
+      e => logError(s"xxx agg fun:$e, ${e.aggregateFunction.aggBufferAttributes}")
+    }
+    val replacedResultExpressions =
+      groupingExpressions ++ aggregateExpressions.map(_.resultAttribute)
+    val agg = CHHashAggregateExecTransformer(
       requiredChildDistributionExpressions,
       groupingExpressions.distinct,
       aggregateExpressions,
       aggregateAttributes,
       initialInputBufferOffset,
-      resultExpressions.distinct,
+      // resultExpressions.distinct,
+      replacedResultExpressions,
       child
     )
+    // val xoutputs = CHHashAggregateExecTransformer.getCHAggregateResultAttributes(
+    //  aggregateExpressions,
+    //  resultExpressions.slice(groupingExpressions.length, resultExpressions.length))
+    // logError(s"xxx adjust agg output: $xoutputs")
+    logError(s"xxx agg output: ${agg.output}")
+    agg
+  }
 
   /** Generate HashAggregateExecPullOutHelper */
   override def genHashAggregateExecPullOutHelper(