[CALCITE-2051] Rules using Aggregate might check for simple grouping sets incorrectly

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 1.15.0
• Component/s: core
• Assignee: Jesús Camacho Rodríguez
• Reporter: Jesús Camacho Rodríguez
• Created: 14/Nov/17 21:57
• Updated: 27/Feb/24 22:24
• Resolved: 20/Nov/17 21:30

Description

CALCITE-1069 removed the indicator columns for Aggregate operators. In some places, the indicator boolean check was replaced by the following check: aggregate.getGroupSets().size() > 1. However, that check is incomplete, it should have been replaced by aggregate.getGroupType() != Group.SIMPLE.

For instance : https://github.com/apache/calcite/blob/master/core/src/main/java/org/apache/calcite/rel/rules/AggregateProjectMergeRule.java#L91

Issue Links

• is depended upon by: CALCITE-2049 Release Calcite 1.15.0