Noemi Pap-Takacs has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22831
Change subject: IMPALA-11672: Update 'transient_lastDdlTime' for Iceberg tables ...................................................................... IMPALA-11672: Update 'transient_lastDdlTime' for Iceberg tables Update 'transient_lastDdlTime' after DDL operations including DROP PARTITION as well. Renaming an Iceberg table is an exception: Iceberg does not keep track of the table name in the metadata files, so there is no Iceberg transaction to change it. The table name is a concept that exists only in the catalog. If we rename the table, we only edit the catalog entry, but nothing changes on the file system, so the change is not visible for Iceberg or an other catalog. Therefore renaming an Iceberg table does not change the 'transient_lastDdlTime' table property because rename is a catalog-level operation for Iceberg tables, and not table-level. Testing: - added Iceberg tests to test_last_ddl_update.py Change-Id: I7e5f63b50bd37c80faf482c4baf4221be857c54b --- M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java M tests/metadata/test_last_ddl_time_update.py 2 files changed, 77 insertions(+), 25 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/22831/1 -- To view, visit http://gerrit.cloudera.org:8080/22831 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I7e5f63b50bd37c80faf482c4baf4221be857c54b Gerrit-Change-Number: 22831 Gerrit-PatchSet: 1 Gerrit-Owner: Noemi Pap-Takacs <npaptak...@cloudera.com>
