Timo Walther created FLINK-26843: ------------------------------------ Summary: Cleanup code around TableConfig/ReadableConfig Key: FLINK-26843 URL: https://issues.apache.org/jira/browse/FLINK-26843 Project: Flink Issue Type: Sub-task Components: Table SQL / Planner Reporter: Timo Walther Assignee: Marios Trivyzas
- Replace usages of TableConfig#getLocalTimeZone() with TableConfigUtils.getLocalTimeZone(tableConfig) to facilitate passing ReadableConfig instead of concrete TableConfig. - Cleanup code by using the utilitty methods of ShortcutUtils: unwrapTableConfig and unwrapContext. Replace TableConfig with ReadableConfig where possible, to cleanup the usages, and make sure that TableConfig is only used where it's absolutely necessary, because of the extra functionality it provides. - Cleanup usages and naming of variables regarding TableConfig and ReadableConfig to make clear what is used/passed as variables and method args. - Cleanup some test code by using PlannerMocks instead of creating PlannerContext manually. - Cleanup some test code by using JsonSerdeTestUtil#configuredSerdeContext() to avoid creating SerdeContexts manually. -- This message was sent by Atlassian Jira (v8.20.1#820001)