add comments for getRowConstructFuncName

apache · Nov 23, 2023 · 811e8f3 · 811e8f3
1 parent 18823d1
commit 811e8f3
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/backends-velox/src/main/scala/io/glutenproject/utils/VeloxIntermediateData.scala b/backends-velox/src/main/scala/io/glutenproject/utils/VeloxIntermediateData.scala
@@ -102,6 +102,11 @@ object VeloxIntermediateData {
     TypeBuilder.makeStruct(false, structTypeNodes.asJava)
   }
 
+  /**
+   * Obtain the name of the RowConstruct function, only decimal avg and sum currently require the
+   * use of row_constructor, while the rest use the Gluten custom modified
+   * row_constructor_with_null.
+   */
   def getRowConstructFuncName(aggFunc: AggregateFunction): String = aggFunc match {
     case _: Average | _: Sum if aggFunc.dataType.isInstanceOf[DecimalType] => "row_constructor"
     case _ => "row_constructor_with_null"