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)
