[ 
https://issues.apache.org/jira/browse/SPARK-32740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17187150#comment-17187150
 ] 

Apache Spark commented on SPARK-32740:
--------------------------------------

User 'c21' has created a pull request for this issue:
https://github.com/apache/spark/pull/29583

> Refactor common partitioning/distribution logic to BaseAggregateExec
> --------------------------------------------------------------------
>
>                 Key: SPARK-32740
>                 URL: https://issues.apache.org/jira/browse/SPARK-32740
>             Project: Spark
>          Issue Type: Improvement
>          Components: SQL
>    Affects Versions: 3.1.0
>            Reporter: Cheng Su
>            Priority: Trivial
>
> For all three different aggregate physical operator: `HashAggregateExec`, 
> `ObjectHashAggregateExec` and `SortAggregateExec`, they have same 
> `outputPartitioning` and `requiredChildDistribution` logic. Refactor these 
> same logic into their super class `BaseAggregateExec` to avoid code 
> duplication and future bugs (similar to `HashJoin` and `ShuffledJoin`).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to