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

Marios Trivyzas commented on FLINK-26190:
-----------------------------------------

Also consider making *PlannerConfiguration* more generic, like 
*MergedConfiguration* and use this new common class for both case.

> Remove TableConfig from ExecNodeConfiguration
> ---------------------------------------------
>
>                 Key: FLINK-26190
>                 URL: https://issues.apache.org/jira/browse/FLINK-26190
>             Project: Flink
>          Issue Type: Technical Debt
>          Components: Table SQL / Planner
>            Reporter: Marios Trivyzas
>            Priority: Major
>
> Currently, *ExecNodeConfiguration* holds *TableConfig* on top of the 
> *plannerConfig* and the *persistedConfig,* since it's needed by the 
> *CodeGeneratorContext* basically for the {*}nullCheck{*}.
> The *nullCheck* should be deprecated and removed which will facilitate the 
> usage of the simple iface: *ReadableConfig* in *CodeGeneratorContext* ** 
> instead of *TableConfig* and the *TableConfig* could be removed 
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to