[
https://issues.apache.org/jira/browse/BEAM-2477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16061796#comment-16061796
]
ASF GitHub Bot commented on BEAM-2477:
--------------------------------------
Github user JingsongLi closed the pull request at:
https://github.com/apache/beam/pull/3398
> BeamAggregationRel should use Combine.perKey instead of GroupByKey
> ------------------------------------------------------------------
>
> Key: BEAM-2477
> URL: https://issues.apache.org/jira/browse/BEAM-2477
> Project: Beam
> Issue Type: Improvement
> Components: dsl-sql
> Reporter: Jingsong Lee
> Assignee: Jingsong Lee
> Labels: dsl_sql_merge
>
> Their semantics are the same, but the efficiency of implementation is quite
> different, and at the runner level there is a lot of optimization for
> `Combine.perKey`.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
