[#351] feat(catalog-lakehouse): make IcebergTableOps#updateTable more convenient

[FANNG1]

What changes were proposed in this pull request?

add a new updateTable interface in IcebergTableOps

Why are the changes needed?

graviton developer could pass TableChange to IcebergTableOps, and no need to care about details of Iceberg

fixes: #351

Does this PR introduce any user-facing change?

No

How was this patch tested?

UT

[github-actions]

Code Coverage Report

Overall Project	63.4% -0.25%	🟢
Files changed	88.29%	🟢

Module	Coverage
catalog-lakehouse	91.65% -6.42%	🟢
api	69.87%	🟢

Files

Module	File	Coverage
catalog-lakehouse	IcebergTableOps.java	90%	🟢
	IcebergTableOpsHelper.java	87.97% -12.03%	🟢
api	TableChange.java	85.09%	🟢

[FANNG1]

@yunqing-wei @jerryshao , this's pr is ready to review

[jerryshao]

@yunqing-wei can you please review firstly?

[Clearvive]

@yunqing-wei can you please review firstly?

Suggestions for modifications were made in the review, and Xiaojing is currently making the changes.

[jerryshao]

I think you should leave all the review comments on the PR, not just sync up off-line.

[Clearvive]

+1

[jerryshao]

If this PR is mainly targeted to add a bunch of helper methods to convert from Graviton's TableChange to Iceberg's UpdateSchema, I would suggest:

1. Writing all these converters to a new so-called adapter class or helper class. Leave the IcebergTableOps here all about the Iceberg operations.
2. Differentiate the parameter name, if the parameter is iceberg's, then using icebergXxx, if from Graviton, use gravitonXxx.

[FANNG1]

1. Writing all these converters to a new so-called adapter class or helper class. Leave the IcebergTableOps here all about the Iceberg operations.

add IcebergTableOpsHelper to transfer from TableChange of Graviton to IcebergTableChange. IcebergCatalogOperations should use the following code to updateTable. cc @yunqing-wei

    IcebergTableChange icebergTableChange =
        icebergTableOpsHelper.makeIcebergTableChanges(gravitonNameIdentifier, gravitonTableChanges);
    return icebergTableOps.updateTable(icebergTableChange);

2. Differentiate the parameter name, if the parameter is iceberg's, then using icebergXxx, if from Graviton, use gravitonXxx.

fixed.

[jerryshao]

There're some style issues, please fix them first.

[FANNG1]

There're some style issues, please fix them first.

done