[CALCITE-2852] RexNode simplification does not traverse unknown functions

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.19.0
• Labels: pull-request-available
• Assignee: Zoltan Haindrich
• Reporter: Zoltan Haindrich
• Created: 15/Feb/19 14:34
• Updated: 25/Mar/19 00:11
• Resolved: 21/Feb/19 05:29

Description

Currently simplification concentrates on basic constructs (and/or/=/...), but if it encounters an UDF which is not known; the recursion is stopped.

For example the following expression is not simplified:

  @Test public void testSimplifyRecurseIntoArithmetics() {
    checkSimplify(
        plus(literal(1),
            case_(
                falseLiteral, literal(1),
                trueLiteral, literal(2),
                literal(3))),
        "+(1, 2)");
  }

Issue Links

• is duplicated by: CALCITE-2862 Deprecating ExprSimplifier causes some operands not being simplified
• links to: GitHub Pull Request #1053