[CALCITE-1166] Disallow sub-classes of RelOptRuleOperand

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.8.0
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 23/Mar/16 22:17
• Updated: 07/Jul/20 06:35
• Resolved: 22/Apr/16 08:10

Description

Disallow sub-classes of RelOptRuleOperand. The only reason that people have sub-classed RelOptRuleOperand is to override the boolean matches(RelNode) method, and you can now achieve that by supplying a Predicate to the constructor.

Next release, we will deprecate RelOptRuleOperand's protected constructor and create a new private or package-protected one. Later we will remove that constructor.

Issue Links

• is related to: CALCITE-3923 Refactor how planner rules are parameterized
• relates to: CALCITE-2832 Create RuleOperandBuilder for building RelOptRule operands