[CALCITE-1436] AggregateNode NPE for aggregators other than SUM/COUNT

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.17.0
• Component/s: core
• Assignee: Michael Mior
• Reporter: Gian Merlino
• Created: 13/Oct/16 16:50
• Updated: 20/Jul/18 08:18
• Resolved: 13/Jun/18 14:43

Description

AggregateNode.getAccumulator does this for any aggregation other than COUNT or SUM:

final AggImpState agg = new AggImpState(0, call, false);
int stateSize = agg.state.size();

This NPEs because "state" is null on freshly created AggImpState instances.

Issue Links

• Blocked: CALCITE-2228 An inevitable NPE while using MAX aggregation function
• is depended upon by: CALCITE-1805 Druid adapter incorrectly pushes down "COUNT(c)"; Druid only supports "COUNT(*)"
• is duplicated by: CALCITE-1437 AggregateNode NPE for aggregators other than SUM/COUNT
• is duplicated by: CALCITE-1687 NPE in AggregateNode.getAccumulator
• is related to: CALCITE-1706 DruidAggregateFilterTransposeRule causes very fine-grained aggregations to be pushed to Druid