[FLINK-26190] Remove getTableConfig from ExecNodeConfiguration

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.15.0
• Component/s: Table SQL / Planner
• Labels: pull-request-available
• Assignee: Dian Fu
• Reporter: Marios Trivyzas
• Created: 16/Feb/22 12:59
• Updated: 08/Apr/22 08:19
• Resolved: 07/Apr/22 14:40

Description

Currently, ExecNodeConfig holds TableConfig instead of ReadableConfig for the configuration coming from the planner, because it's used by

CommonPythonUtil#getMergedConfig. This should be fixed, so that CommonPythonUtil#getMergedConfig cam use a ReadableConfig instead, and then we can pass the ExecNodeConfig which holds the complete view of Planner's TableConfig + the ExecNode's persistedConfig.

 

To achieve that the getMergedConfig methods of PythonConfigUtil must be changed, and also the temp solution in 

PythonFunctionFactory#getPythonFunction must be changed as well:

if (config instanceof TableConfig) {
    PythonDependencyUtils.merge(mergedConfig, ((TableConfig) config).getConfiguration());
} else {
    PythonDependencyUtils.merge(mergedConfig, (Configuration) config);
}

Issue Links

• relates to: FLINK-26194 Deprecate unused options in TableConfig
• links to: GitHub Pull Request #19333