[CALCITE-3756] RelSubset should not match operand(RelNode.class)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.22.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Haisheng Yuan
• Created: 23/Jan/20 20:57
• Updated: 06/Mar/20 04:35
• Resolved: 29/Jan/20 19:39

Description

Even RelSubset is a subclass of RelNode, but they are quite different. RelNode already fires these rules, it is not necessary to fire for RelSubset. More importantly, it never worked due to CALCITE-3755.
In addition, it will waste time to try to iterate class operands of RelSubset and try to match pattern operand(RelNode), given that it never matches when ascending.

Issue Links

• links to: GitHub Pull Request #1773