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

Benchao Li commented on CALCITE-5381:
-------------------------------------

This is an interesting issue. This makes me thinking about what the 
compatibility should theĀ {{RelBuilder}} guarantee. Currently it seems that we 
only care about the "API compatibility", but not "result compatibility".

I believe that there are more cases that we changed the result for some methods 
of {{RelBuilder}} without considering compatibility problems.

[~Sergey Nuyanzin] Could you kindly elaborate a little more about the situation 
you are in now? Or linking to a Flink issue would help a lot for us. I'm 
thinking that whether we can use some other methods instead of 
{{RelBuilder#join}}, e.g. {{RelBuilder#correlate}}?

> Add configuration via property to turn on/off check if correlated in 
> RelBuilder
> -------------------------------------------------------------------------------
>
>                 Key: CALCITE-5381
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5381
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Sergey Nuyanzin
>            Priority: Major
>
> The issue is that the change done in 
> https://issues.apache.org/jira/browse/CALCITE-4668 is not supported now in 
> Flink and requires some significant efforts to make it working. 
> At the same time we would like to move forward with Calcite update.
> Currently there is no way to turn this off via property e.g. or via 
> overriding of methods.
> This jira issues proposes introducing such property (turn on by default)
> turning off means Calcite 1.27.0 behavior



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to