[ https://issues.apache.org/jira/browse/CALCITE-3668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17009027#comment-17009027 ]
Vladimir Sitnikov commented on CALCITE-3668: -------------------------------------------- {quote} But I still think the method should be called inside VolcanoRuleCall instead of rule itself, therefore getSubsets doesn't need to be public{quote} That probably would work. Would you update the PR and add the relevant test? > VolcanoPlanner doesn't match all the RelSubSet in matchRecursive > ---------------------------------------------------------------- > > Key: CALCITE-3668 > URL: https://issues.apache.org/jira/browse/CALCITE-3668 > Project: Calcite > Issue Type: Bug > Reporter: Haisheng Yuan > Priority: Major > Labels: pull-request-available > Time Spent: 20m > Remaining Estimate: 0h > > If I have a rule to match pattern with Filter-RelSubset, VolcanoPlanner only > matches 1 RelSubset in the RelSet, instead of all the subsets. > {code:java} > if (operand.getMatchedClass() == RelSubset.class) { > // If the rule wants the whole subset, we just provide it > successors = ImmutableList.of(subset); > } else { > successors = subset.getRelList(); > } > {code} -- This message was sent by Atlassian Jira (v8.3.4#803005)