[ 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)