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

Laurent Goujon commented on CALCITE-3111:
-----------------------------------------

{quote}
I guess it would be convenient to generalize RelDecorrelator to work with other 
kind of RelNodes (apart from Logical ones). Correct if I am wrong but this 
applies to all relational expressions used in the decorrelator (not only 
Correlate and LogicalCorrelate).
{quote}

That's the idea: refactor {{RelDecorrelator}} to use {{RelBuilder}} + base 
Calcite classes ({{Project}}, {{Filter}}, {{Correlate}}, ...) instead of 
{{LogicalXXX}} nodes directly...

> Allow custom implementations of Correlate in RelDecorrelator
> ------------------------------------------------------------
>
>                 Key: CALCITE-3111
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3111
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Juhwan Kim
>            Assignee: Juhwan Kim
>            Priority: Minor
>
> Currently, RelDecorrelator code only works for LogicalCorrelate. 
> Decorrelating through Calcite would become much more flexible if it allows 
> using custom implementations of Correlate. This would require refactoring all 
> logical rels used in RelDecorrelator to the abstract ones(e.g 
> LogicalCorrelate -> Correlate).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to