[CALCITE-3908] JoinCommuteRule should update all input references in join condition

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 1.23.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Jesus Camacho Rodriguez
• Reporter: Jesus Camacho Rodriguez
• Created: 09/Apr/20 17:51
• Updated: 24/May/20 11:59
• Resolved: 10/Apr/20 01:09

Description

JoinCommuteRule swaps the inputs of a join. It relies on an internal class VariableReplacer to update the references in the join condition. However, this class does not extend RexShuttle and ends up ignoring some of the references, e.g., those in RexFieldAccess.

Issue Links

• links to: GitHub Pull Request #1906