[CALCITE-1661] Support aggregation functions on DECIMAL in DruidAdapter

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.12.0
• Fix Version/s: 1.12.0
• Component/s: druid
• Assignee: Jesus Camacho Rodriguez
• Reporter: Jesus Camacho Rodriguez
• Created: 27/Feb/17 09:47
• Updated: 24/Mar/17 03:20
• Resolved: 03/Mar/17 11:49

Description

Currently, whether to use fractional or integer aggregations is based on following code (L699 in DruidQuery.java).

final boolean b = aggCall.getType().getSqlTypeName() == SqlTypeName.DOUBLE;

Since Hive might use other fractional types for the aggregation, we might end up using the wrong type of aggregation in Druid. We could extend the check as follows:

final boolean b = SqlTypeName.FRACTIONAL_TYPES.contains(aggCall.getType().getSqlTypeName());

Issue Links

• is depended upon by: CALCITE-1657 Release Calcite 1.12.0