[BEAM-2477] BeamAggregationRel should use Combine.perKey instead of GroupByKey

• Type: Improvement
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: Not applicable
• Component/s: dsl-sql
• Labels: Done dsl_sql_merge
• Assignee: Jingsong Lee
• Reporter: Jingsong Lee
• Created: 20/Jun/17 01:08
• Updated: 24/Jul/20 20:20
• Resolved: 24/Jun/17 10:39

Description

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`.

Issue Links

• links to: GitHub Pull Request #3398