[ 
https://issues.apache.org/jira/browse/CALCITE-4650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17363509#comment-17363509
 ] 

Vladimir Ozerov commented on CALCITE-4650:
------------------------------------------

Hi [~zabetak], thanks for noting this. I fixed the version in this and a couple 
of other tickets.

> TopDownRuleDriver.DeriveTrait.derive produces wrong AssertionError when 
> registering new nodes
> ---------------------------------------------------------------------------------------------
>
>                 Key: CALCITE-4650
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4650
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.27.0
>            Reporter: Vladimir Ozerov
>            Assignee: Vladimir Ozerov
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.28.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> {{TopDownRuleDriver.DeriveTrait.derive}} asserts that a derived node belongs 
> to the same {{RelSet}} as the original node. 
> If the original node {{N1}} refers to set {{S1}}, the derived node {{N2}} 
> might be registered in the set {{S2}}, if {{S1}} was merged to {{S2}} between 
> the registration of {{N1}} and derivation of {{N2}}. 
> Currently, we compare the original set of the original node {{N1}} with the 
> canonical set of the derived node {{N2}}. This leads to false assertions.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to