Quanlong Huang has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/22036


Change subject: IMPALA-13518: Show target name of COMMIT_TXN events in logs
......................................................................

IMPALA-13518: Show target name of COMMIT_TXN events in logs

The message of a COMMIT_TXN event just contains the transaction id
(txnid). In the logs of top-10 expensive events and top-10 targets that
contribute to the lag, we show the target as CLUSTER_WIDE.

However, when processing the events, catalogd actually finds
the involved tables and reloads them. It'd be helpful to show the names
of the tables involved in the transaction.

This patch overrides the getTargetName() method in CommitTxnEvent to
show the table names. They are collected after the event is processed.

Tests:
 - TODO: Run CORE tests and verify the logs

Change-Id: I4a7cb5e716453290866a4c3e74c0d269f621144f
---
M fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java
M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
2 files changed, 13 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/36/22036/1
--
To view, visit http://gerrit.cloudera.org:8080/22036
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4a7cb5e716453290866a4c3e74c0d269f621144f
Gerrit-Change-Number: 22036
Gerrit-PatchSet: 1
Gerrit-Owner: Quanlong Huang <[email protected]>

Reply via email to