[CALCITE-1781] Allow expression in CUBE and ROLLUP

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.13.0
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 09/May/17 18:39
• Updated: 27/Feb/24 22:23
• Resolved: 10/May/17 17:26

Description

Currently we only allow columns as arguments to CUBE and ROLLUP operators, whereas we allow expressions in the GROUP BY clause. This change would also allow expressions as arguments to CUBE and ROLLUP. Example:

// the following are valid, and will remain valid
select deptno from emp group by deptno; 
select deptno +1 from emp group by deptno + 1;
select deptno from emp group by cube(deptno);
select deptno from emp group by rollup(deptno);

// the following are currently invalid, but would be valid
select deptno +1 from emp group by cube(deptno + 1, empno - 1);
select deptno +1 from emp group by rollup(deptno + 1, empno - 1);

Note that as of CALCITE-1306, and with SqlConformance.isGroupByOrdinal enabled, select deptno from emp group by cube(1) should be valid because 1 is interpreted as an ordinal in the SELECT clause, not an expression.

Issue Links

• is related to: CALCITE-1306 Allow GROUP BY and HAVING to reference SELECT expressions by ordinal and alias