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

Haisheng Yuan commented on CALCITE-3111:
----------------------------------------

It is so sad that Calcite allow optimization directly on physical operators, 
which will bind itself. I don't think we should unlash the shackles to allow 
decorrelating directly on physical operators. More freedom doesn't make it 
stronger. But if [~Juhwan] wants to provides his own logical correlate 
operator, that would be another story. I would like to know in what aspect the 
current LogicalCorrelate doesn't meet your requirement, maybe we can extend 
that.

> 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