[CALCITE-2862] Deprecating ExprSimplifier causes some operands not being simplified

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Component/s: core
• Assignee: Unassigned
• Reporter: Rong Rong
• Created: 23/Feb/19 15:57
• Updated: 19/Sep/19 19:18
• Resolved: 23/Feb/19 17:07

Description

This might not be a bug or the original intention of CALCITE-2726 since it is not part of the PR.
The reduction of ExprSimplifier (RexShuttle) actually causes some of the chained operations not being simplified. For example:

AS(
  +(
    someAggFunc($2, $0), 
    CAST(1):BIGINT NOT NULL
    ),
  someName'
)

Somehow did not get simplified (e.g. the CAST(1) still remains instead of being simplified as a Literal.

My initial feeling is that AS is not one of the kinds in the case-switch code in RexSimplify.

Issue Links

• duplicates: CALCITE-2852 RexNode simplification does not traverse unknown functions