[
https://issues.apache.org/jira/browse/CALCITE-3668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17009027#comment-17009027
]
Vladimir Sitnikov edited comment on CALCITE-3668 at 1/6/20 5:25 PM:
--------------------------------------------------------------------
{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 would probably work.
Would you please update the PR and add the relevant test?
was (Author: vladimirsitnikov):
{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 please 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)
