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>

Reply via email to