[flink] Avoid deprecated usage on TableSchema, DataType and DescriptorProperties

[yunfengzhou-hub]

Purpose

Linked issue: #4442

This PR removes deprecated usages about Flink Table API, mainly including TableSchema and DescriptorProperties.

Tests

Existing test cases are used to verify this change.

API and Format

This change does not affect API or storage format.

Documentation

This change does not affect introduce a new feature.

[JingsongLi]

I am concern about the serialization for watermark spec, can you create some test for compatibility? (You can create schema file from old Paimon, and read it in tests)

[yunfengzhou-hub]

I am concern about the serialization for watermark spec, can you create some test for compatibility? (You can create schema file from old Paimon, and read it in tests)

@JingsongLi FlinkCatalogPropertiesUtilTest has recorded the serialization result as follows

Map<String, String> expected = new HashMap<>();
String watermarkPrefix = compoundKey(SCHEMA, WATERMARK, 0);
expected.put(compoundKey(watermarkPrefix, WATERMARK_ROWTIME), "test_time");
expected.put(
        compoundKey(watermarkPrefix, WATERMARK_STRATEGY_EXPR),
        "`test_time` - INTERVAL '0.001' SECOND");
expected.put(compoundKey(watermarkPrefix, WATERMARK_STRATEGY_DATA_TYPE), "TIMESTAMP(3)");

This code stands for the serialization result since the older versions of Paimon. This PR does not change this part of code, which means the compatibility should be guaranteed.

[JingsongLi]

Why need to copy it? I see there is no copy in DefaultCatalogTable.

[yunfengzhou-hub]

The copy was added because the copy had been made in the existing implementation. But I agree with it that the copy is unnecessary. I'll remove it.

[JingsongLi]

+1