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

Danny Chen commented on CALCITE-4225:
-------------------------------------

I'm planning to merge within 24 hours ~

> Make RelDecorrelator pluggable
> ------------------------------
>
>                 Key: CALCITE-4225
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4225
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.25.0
>            Reporter: Danny Chen
>            Assignee: Danny Chen
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.27.0
>
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> {{RelDecorrelator}} is our core component that decorrelates the queries. But 
> actually, the pattern it can decorrelate successfully is very limited. And it 
> often causes bug because it decorrelates into a wrong plan (with non-correct 
> sementics).
> When there are bugs there, the downstream project can only wait for the 
> version upgrade (if it is fixed in Calcite master), or they copy the whole 
> Java class and override the Calcite one which is hard to maintain.
> It would be nice if we can make the decorrelation logic pluggable. e.g. we 
> can control how a RelNode was decorrelated(not a coarse-grained flag to 
> turn-on/off the decorrelation).



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

Reply via email to