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

Stamatis Zampetakis commented on CALCITE-5381:
----------------------------------------------

Hey [~Sergey Nuyanzin], apologies for the late response. If there is no other 
way than to modify the RelBuilder to overcome the problems that appeared in 
Flink, I would pick the least intrusive change with fewer possible 
side-effects. I haven't thought this through to choose between config property/ 
protected method/ or something else. I am leaving the choice up to you but make 
sure to include tests covering the new code path if any. 

> 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