dianfu commented on issue #9874: [FLINK-14240][table] Merge table config parameters(TableConfig#getConfiguration) into global job parameters(ExecutionConfig#getGlobalJobParameters) when running with the legacy planner. URL: https://github.com/apache/flink/pull/9874#issuecomment-544393616 Hi @twalthr, Thanks a lot for your reply. Regarding adding the `global job configuration`, as it will introduce new API interfaces and it will be better to discuss it in the ML or the mentioned FLIPs. The aim of this PR is to align the legacy planner with the blink planner(maybe we should change the title of this PR to make it more clear). So how about limiting the scope of this PR to aligning the two planners and discussing the API changes in the FLIP discussion? Best, Dian
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
