[jira] [Commented] (CALCITE-5381) Add configuration via property to turn on/off check if correlated in RelBuilder

["Sergey Nuyanzin (Jira)" <ji...@apache.org>]

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