You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flink.apache.org by "Timo Walther (Jira)" <ji...@apache.org> on 2022/03/24 09:24:00 UTC

[jira] [Created] (FLINK-26843) Cleanup code around TableConfig/ReadableConfig

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)