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

Sergey Nuyanzin commented on CALCITE-5381:
------------------------------------------

[~zabetak] I got your point about config flag.

Since anyway Flink extends RelBuilder do you see any issue if we make 
{{RelBuilder#checkIfCorrelated}} extendable ( {{protected}} instead of 
{{private static}} )?

 

> 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