[ 
https://issues.apache.org/jira/browse/CALCITE-4650?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Haisheng Yuan updated CALCITE-4650:
-----------------------------------
    Description: 
{{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.

Dicussion:
https://lists.apache.org/thread.html/rd26b7a0f55906cebf5abcaf349ab2bec414e164fa4d9f63789bf34c8%40%3Cdev.calcite.apache.org%3E

  was:
{{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.


> 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: 50m
>  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.
> Dicussion:
> https://lists.apache.org/thread.html/rd26b7a0f55906cebf5abcaf349ab2bec414e164fa4d9f63789bf34c8%40%3Cdev.calcite.apache.org%3E



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

Reply via email to