xupefei commented on code in PR #46245: URL: https://github.com/apache/spark/pull/46245#discussion_r1596845911
########## sql/core/src/main/scala/org/apache/spark/sql/expressions/Aggregator.scala: ########## @@ -49,6 +49,7 @@ import org.apache.spark.sql.execution.aggregate.TypedAggregateExpression * @tparam OUT The type of the final output result. * @since 1.6.0 */ +@SerialVersionUID(2093413866369130093L) Review Comment: It's for mapping the client's `Aggregator` class to this one. This is required because we now serialise the whole `Aggregator` instance on the client side. If we decide to serialise individual methods instead, this `SerialVersionUID` can be abolished. Wdyt? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org