[CALCITE-3955] Remove the first operand of RexCall from SqlWindowTableFunction

• Type: Bug
• Status: Reopened
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.22.0
• Component/s: core
• Assignee: Danny Chen
• Reporter: Danny Chen
• Created: 24/Apr/20 09:40
• Updated: 11/May/20 22:08

Description

In CALCITE-3382, we introduced TUMBLE window function to replace the deprecated group tumble window.

But for query

select *
from table(tumble(table Shipments, descriptor(rowtime), INTERVAL '1' MINUTE))

the outputs plan is

LogicalProject(ORDERID=[$0], ROWTIME=[$1], window_start=[$2], window_end=[$3])
  LogicalTableFunctionScan(invocation=[TUMBLE($1, DESCRIPTOR($1), 60000:INTERVAL MINUTE)], rowType=[RecordType(INTEGER ORDERID, TIMESTAMP(0) ROWTIME, TIMESTAMP(0) window_start, TIMESTAMP(0) window_end)])
    LogicalProject(ORDERID=[$0], ROWTIME=[$1])
      LogicalTableScan(table=[[CATALOG, SALES, SHIPMENTS]])

The first operand of TUMBLE rex call is always the last input field, but actually it represents the source table which is the input rel node.

This issue remove the first operand from the RexCall because it is useless and confusing.

Issue Links

• is a child of: CALCITE-3271 Table-valued function windowing and EMIT syntax support in Calcite
• is related to: CALCITE-3382 Rename current TUMBLE to "$TUMBLE" and add TUMBLE to Parser
• relates to: CALCITE-3970 Table-valued function TUMBLE uses non-standard syntax