[CALCITE-1172] Add rule to flatten two Aggregate operators into one - ASF JIRA

Voters

Watch issue

Watchers

Link

Clone

Update Comment Author

Replace String in Comment

Update Comment Visibility

Delete Comments

XML

Word

Printable

JSON

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: None
Fix Version/s: 1.20.0
Component/s: None
Labels:
- pull-request-available

Description

If there is an Aggregate on top of an Aggregate, and the top aggregate's group key is a subset of the lower aggregate's group key, and the aggregates are expansions of rollups, then a rule would convert into a single aggregate.

For example, SUM of SUM becomes SUM; SUM of COUNT becomes COUNT; MAX of MAX becomes MAX; MIN of MIN becomes MIN. AVG of AVG would not match, nor would COUNT of COUNT.

The rule would convert

Aggregate({x}, SUM(c1), MIN(c2), MAX(c3))
  Aggregate({x, y}, SUM(c1), MIN(c2), MAX(c3))
    R

into

Aggregate({x}, SUM(c1), MIN(c2), MAX(c3))
  R

The rule would not work if there was anything between the two Aggregate operators.

The implementation would use SqlSplittableAggFunction to deduce rollups, and in fact would be practically the converse of AggregateExchangeTransposeRule proposed in CALCITE-907.