[CALCITE-5949] RexExecutable should return unchanged original expressions when it fails

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.35.0
• Fix Version/s: 1.36.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Ruben Q L
• Reporter: Claude Brisson
• Created: 22/Aug/23 10:21
• Updated: 10/Nov/23 11:34
• Resolved: 27/Oct/23 07:15

Description

While reducing, when encountering an invalid expression in the list of constant expressions, RexExecutor is meant to return all initial expressions unchanged.

It fails to do so, because already handled correct expressions have already been added to the returned list, which can be greater than the input list.

For instance, when given the list { LN(2), LN(-2) }, the RexExecutor will output a list of length 3.

Issue Links

• links to: GitHub Pull Request #3390
• links to: GitHub Pull Request #3486