Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22653 to look at the new patch set (#3). Change subject: IMPALA-13884: Add more details in metadata loading logs ...................................................................... IMPALA-13884: Add more details in metadata loading logs Adds the target of the event in HMS event processing logs. For AlterPartitionEvent, the partition name will also be printed. Note that Add/DropPartitionEvent could have multiple partitions so ignore them for now. E.g. EventId: 28781 EventType: ALTER_TABLE Target: tpcds.store_sales EventId: 5271 EventType: ALTER_PARTITION Target: functional.alltypes year=2009/month=1 In the reason of loading table metadata, adds the event id if the request comes from the EventProcessor. E.g. Reloading partition metadata: functional_json.decimal_rtf_tiny_tbl dpc=1 (INSERT event 15685) Reloading metadata for table definition and all partition(s) of functional_json.manynulls (INSERT event 15765) For logs of execDdl requests on AlterTable operations, add the AlterType. Adds a log when start recovering partitions. Also adds an overload of AcidUtils.isTransactionalTable() for FeTable to simplify some codes. For the warning of waiting for table write lock too long, also logs the stacktrace. Tests: - Ran CORE tests Change-Id: I204d5922e055fd8501b5573e3b913f8874d891d6 --- M fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java M fe/src/main/java/org/apache/impala/analysis/Analyzer.java M fe/src/main/java/org/apache/impala/analysis/ConvertTableToIcebergStmt.java M fe/src/main/java/org/apache/impala/analysis/ResetMetadataStmt.java M fe/src/main/java/org/apache/impala/analysis/StmtMetadataLoader.java M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/org/apache/impala/catalog/FeCatalogUtils.java M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java M fe/src/main/java/org/apache/impala/service/Frontend.java M fe/src/main/java/org/apache/impala/util/AcidUtils.java M fe/src/main/java/org/apache/impala/util/CatalogOpUtil.java M fe/src/test/java/org/apache/impala/util/CatalogOpUtilTest.java 14 files changed, 104 insertions(+), 79 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/53/22653/3 -- To view, visit http://gerrit.cloudera.org:8080/22653 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I204d5922e055fd8501b5573e3b913f8874d891d6 Gerrit-Change-Number: 22653 Gerrit-PatchSet: 3 Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>