[CALCITE-3853] Minor improvements in SortJoinCopyRule

• Type: Task
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.22.0
• Fix Version/s: 1.23.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Ruben Q L
• Reporter: Ruben Q L
• Created: 11/Mar/20 15:42
• Updated: 24/May/20 12:00
• Resolved: 12/Mar/20 13:04

Description

A few improvements can be made on SortJoinCopyRule (introduced via CALCITE-2624):

• The default INSTANCE currently matches LogicalSort + Join, which seems inconsistent, it should match LogicalSort + LogicalJoin
• The constructor should be public, in order to allow clients to create their own version of the rule

Issue Links

• is related to: CALCITE-2624 Add a rule to copy a sort below a join operator
• links to: GitHub Pull Request #1857