[CALCITE-3927] RelSubset is not fired for rule when set gets merged

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.23.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Haisheng Yuan
• Created: 15/Apr/20 18:35
• Updated: 24/May/20 12:00
• Resolved: 21/Apr/20 18:29

Description

In VolcanoPlanner, when set gets merged, planner fires rules again for RelNodes in both sets, but not for RelSubset. We might miss something because of this.

If all the logical transformation rules and physical implementation rules are separated out in different stage and physical rules don't do logical work, we might be OK. But the reality is that all the things are mixed together at the moment.

Issue Links

• links to: GitHub Pull Request #1922