[#2927] Improvement(catalog-lakehouse-iceberg): Support more file formats in using clause when create iceberg tables

[caican00]

What changes were proposed in this pull request?

Support more file formats in using clause when create iceberg tables, such as using parquet, using orc, using avro, using iceberg.

using other provider will throw an exception.

Why are the changes needed?

Because Iceberg official supports using parquet/orc/avro/iceberg when create a new table.

Fix: #2927

Does this PR introduce any user-facing change?

Yes, users can using parquet/orc/avro/iceberg keywork in using clause when create a new table.

How was this patch tested?

New IT.

[caican00]

Hi @FANNG1 could you help review this PR when you are free? Thanks

[caican00]

all comments have been addressed.

[FANNG1]

LGTM, except one minor comment

[caican00]

LGTM, except one minor comment

new comments have been addressed.

[FANNG1]

LGTM, except one minor comment

new comments have been addressed.

After another thought, seems we forgot to consider the alter table properties action. I'm not sure whether it works for changing Iceberg table provider, could you add provider property as stringImmutablePropertyEntry to IcebergTablePropertiesMetadata#propertyEntries to make it immutable?

[caican00]

LGTM, except one minor comment

new comments have been addressed.

After another thought, seems we forgot to consider the alter table properties action. I'm not sure whether it works for changing Iceberg table provider, could you add provider property as stringImmutablePropertyEntry to IcebergTablePropertiesMetadata#propertyEntries to make it immutable?

ok, i will do this.
because iceberg supports modifying the table file format and users can modify the table file format in Altertable command using write.format.default instead of provider.

[caican00]

LGTM, except one minor comment

new comments have been addressed.

After another thought, seems we forgot to consider the alter table properties action. I'm not sure whether it works for changing Iceberg table provider, could you add provider property as stringImmutablePropertyEntry to IcebergTablePropertiesMetadata#propertyEntries to make it immutable?

ok, i will do this. because iceberg supports modifying the table file format and users can modify the table file format in Altertable command using write.format.default instead of provider.

updated. and added test code.

[FANNG1]

@caican00 merged to main, thanks for your work