[ https://issues.apache.org/jira/browse/FLINK-16835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17069219#comment-17069219 ]
Jark Wu commented on FLINK-16835: --------------------------------- I think {{TableConfig}} is still useful because it can provide some handy methods to set frequently used configurations with proper Javadocs, e.g {{setLocalTimeZone}}, {{setMaxGeneratedCodeLength}}. > Replace TableConfig with Configuration > -------------------------------------- > > Key: FLINK-16835 > URL: https://issues.apache.org/jira/browse/FLINK-16835 > Project: Flink > Issue Type: Improvement > Components: Table SQL / API > Reporter: Timo Walther > Priority: Major > > In order to allow reading and writing of configuration from a file or > string-based properties. We should consider removing {{TableConfig}} and > fully rely on a Configuration-based object with {{ConfigOptions}}. > This effort was partially already started which is why > {{TableConfig.getConfiguration}} exists. > However, we should clarify if we would like to have control and traceability > over layered configurations such as {{flink-conf,yaml < > StreamExecutionEnvironment < TableEnvironment < Query}}. Maybe the > {{Configuration}} class is not the right abstraction for this. -- This message was sent by Atlassian Jira (v8.3.4#803005)