[FLINK-34239] Introduce a deep copy method of SerializerConfig for merging with Table configs in org.apache.flink.table.catalog.DataTypeFactoryImpl

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Done
• Affects Version/s: 1.19.0
• Fix Version/s: 1.20.0
• Component/s: API / Core
• Labels: pull-request-available
• Assignee: Kumar Mallikarjuna
• Reporter: Zhanghao Chen
• Created: 26/Jan/24 03:30
• Updated: 07/Jun/24 10:04
• Resolved: 07/Jun/24 10:04

Description

Problem

Currently, org.apache.flink.table.catalog.DataTypeFactoryImpl#createSerializerExecutionConfig will create a deep-copy of the SerializerConfig and merge Table config into it. However, the deep copy is done by manully calling the getter and setter methods of SerializerConfig, and is prone to human errors, e.g. missing copying a newly added field in SerializerConfig.

Proposal

Introduce a deep copy method for SerializerConfig and replace the curr impl in org.apache.flink.table.catalog.DataTypeFactoryImpl#createSerializerExecutionConfig.

Issue Links

• links to: GitHub Pull Request #24544