Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17248 )
Change subject: IMPALA-10624: TestIcebergTable::test_alter_iceberg_tables failed by stale file format ...................................................................... IMPALA-10624: TestIcebergTable::test_alter_iceberg_tables failed by stale file format During table creation we pushed all the table properties to the Iceberg table, e.g. 'iceberg.file_format'. This works well with current Iceberg version, but causes bugs with newer versions. Newer versions of Iceberg refresh the HMS properties from the table properties for every table update. That means if we intially set a table property, we cannot modify it later because Iceberg will rewrite it with the old value on every table update. These table properties are not needed at the Iceberg table level, so we can just skip passing them during table creation. Testing: * ran existing tests * ran tests with newer version Iceberg * checked manually that we don't set these properties at the Iceberg table level Change-Id: Iff8d5d1d90444aba11d47cfce522aaa45a4a74cc Reviewed-on: http://gerrit.cloudera.org:8080/17248 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java M fe/src/main/java/org/apache/impala/service/IcebergCatalogOpExecutor.java 2 files changed, 32 insertions(+), 4 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/17248 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iff8d5d1d90444aba11d47cfce522aaa45a4a74cc Gerrit-Change-Number: 17248 Gerrit-PatchSet: 4 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: wangsheng <[email protected]>
