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

Stamatis Zampetakis commented on CALCITE-3586:
----------------------------------------------

I would suggest to always use the cluster of the \{{RelBuilder}} when creating 
the new expressions. Like this we could also partially support CALCITE-1681 for 
copying plans between clusters.

> ToLogicalConverter should not create plans with nodes in different clusters 
> ----------------------------------------------------------------------------
>
>                 Key: CALCITE-3586
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3586
>             Project: Calcite
>          Issue Type: Bug
>    Affects Versions: 1.21.0
>            Reporter: Stamatis Zampetakis
>            Assignee: Stamatis Zampetakis
>            Priority: Minor
>
> Currently the {{ToLogicalConverter}} sometimes obtains the cluster from 
> another RelNode directly and sometimes it uses the cluster that is present in 
> the {{RelBuilder}}. Depending on how it is used, these clusters are not 
> necessarily the same leading to plans which may have nodes in different 
> clusters which in most cases is undesirable. 



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

Reply via email to