(incubator-gluten) branch main updated: [VL] Remove useless agg mode check in applyExtractStruct (#5161)

[ph...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

philo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 2eff2ddd0 [VL] Remove useless agg mode check in applyExtractStruct (#5161)
2eff2ddd0 is described below

commit 2eff2ddd0ec25f6a1c887b2da1270ab2a7fa1b25
Author: Joey <jo...@alibaba-inc.com>
AuthorDate: Fri Mar 29 14:02:35 2024 +0800

    [VL] Remove useless agg mode check in applyExtractStruct (#5161)
---
 .../io/glutenproject/execution/HashAggregateExecTransformer.scala    | 5 -----
 1 file changed, 5 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
index cbe0beab2..f71509d5c 100644
--- a/backends-velox/src/main/scala/io/glutenproject/execution/HashAggregateExecTransformer.scala
+++ b/backends-velox/src/main/scala/io/glutenproject/execution/HashAggregateExecTransformer.scala
@@ -140,11 +140,6 @@ abstract class HashAggregateExecTransformer(
     }
 
     for (expr <- aggregateExpressions) {
-      expr.mode match {
-        case Partial | PartialMerge =>
-        case _ =>
-          throw new GlutenNotSupportException(s"${expr.mode} not supported.")
-      }
       val aggFunc = expr.aggregateFunction
       aggFunc match {
         case _ @VeloxIntermediateData.Type(veloxTypes: Seq[DataType]) =>


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org