[CALCITE-3535] EnumerableJoinRule: remove unnecessary Filter on top of INNER Join

• Type: Task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.21.0
• Fix Version/s: 1.22.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Ruben Q L
• Reporter: Ruben Q L
• Created: 25/Nov/19 15:53
• Updated: 06/Mar/20 04:33
• Resolved: 12/Dec/19 10:51

Description

With the implementation of CALCITE-2973, now EnumerableHashJoin supports all type of conditions (not just equi joins). However, there is still one TODO in EnumerableHashJoinRule that, in case of an INNER Join, creates a Filter with the non-equi conditions on top of the EnumerableHashJoin (created only with the equi-conditions), this filter is not really needed, since now EnumerableHashJoinRule can support the full condition: equi and non-equi items.

Issue Links

• is blocked by: CALCITE-3546 Improve EnumerableDefaults nested loop join
• is related to: CALCITE-2973 Allow theta joins that have equi conditions to be executed using a hash join algorithm
• relates to: CALCITE-3598 EnumerableTableScan: wrong JavaRowFormat for elementType String
• links to: GitHub Pull Request #1616