[CALCITE-3763] RelBuilder.aggregate should prune unused fields from the input, if the input is a Project

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.22.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 31/Jan/20 19:25
• Updated: 06/Mar/20 04:34
• Resolved: 14/Feb/20 00:21

Description

RelBuilder.aggregate should prune unused fields from the input, if the input is a Project.

Pruning fields during the planning process is desirable, but often cannot do it - we are applying a RelOptRule that has to return the same fields, or we don't want to add an extra Project do so the pruning. But when we are in RelBuilder.aggregate and the input is a Project, neither of those limitations apply. We already have a Project, we are just making it narrower; and we know what fields the Aggregate will produce.

For example,

SELECT deptno, SUM(sal) FILTER (WHERE b)
FROM (
  SELECT deptno, empno + 10, sal, job = 'CLERK' AS b
  FROM emp)
GROUP BY deptno

becomes

SELECT deptno, SUM(sal) FILTER (WHERE b)
FROM (
  SELECT deptno, sal, job = 'CLERK' AS b
  FROM emp)
GROUP BY deptno

If there are no fields used, remove the Project. (A RelNode with no fields is not allowed.)

SELECT COUNT(*) AS C
FROM (
 SELECT deptno, empno + 10, sal, job = 'CLERK' AS b
 FROM emp)

becomes

SELECT COUNT(*) AS c
FROM emp

Add an option RelBuilder.Config.pruneInputOfAggregate, default true, so that people can disable this rewrite if it causes problems.

Issue Links

• links to: GitHub Pull Request #1806