[CALCITE-3146] Support the detection of nested aggregations for JdbcAggregate in SqlImplementor.

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.19.0
• Fix Version/s: 1.21.0
• Component/s: core
• Labels: pull-request-available
• Assignee: TANG Wen-hui
• Reporter: TANG Wen-hui
• Created: 25/Jun/19 13:30
• Updated: 10/Sep/19 21:37
• Resolved: 01/Jul/19 02:38

Description

The detection of nested aggregations in SqlImplementor.java is only for 

LogicalAggregate:

 

if (rel instanceof LogicalAggregate
    && !dialect.supportsNestedAggregations()
    && hasNestedAggregations((LogicalAggregate) rel)) {
  needNew = true;
}

It should also support other sub-class of Aggregate such as JdbcAggregate .

 

Issue Links

• links to: GitHub Pull Request #1278