maropu commented on a change in pull request #28869:
URL: https://github.com/apache/spark/pull/28869#discussion_r442764355
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/SortAggregateExec.scala
##
@@ -49,7 +49,7 @@ case class SortAggregateExec(
maropu commented on a change in pull request #28869:
URL: https://github.com/apache/spark/pull/28869#discussion_r442761506
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/BaseAggregateExec.scala
##
@@ -53,15 +53,24 @@ trait BaseAggregateExec exte
maropu commented on a change in pull request #28869:
URL: https://github.com/apache/spark/pull/28869#discussion_r442761087
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/BaseAggregateExec.scala
##
@@ -53,15 +53,24 @@ trait BaseAggregateExec exte
maropu commented on a change in pull request #28869:
URL: https://github.com/apache/spark/pull/28869#discussion_r442757181
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
##
@@ -140,7 +140,7 @@ case class Aggregate
maropu commented on a change in pull request #28869:
URL: https://github.com/apache/spark/pull/28869#discussion_r442756818
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/BaseAggregateExec.scala
##
@@ -53,15 +53,24 @@ trait BaseAggregateExec exte
maropu commented on a change in pull request #28869:
URL: https://github.com/apache/spark/pull/28869#discussion_r442753564
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/BaseAggregateExec.scala
##
@@ -53,15 +53,24 @@ trait BaseAggregateExec exte