Hello Bharath Vissapragada, Philip Zeyliger,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/11228
to review the following change.
Change subject: IMPALA-7450. Set thread name during refresh/load operations
......................................................................
IMPALA-7450. Set thread name during refresh/load operations
This adds a small utility class for annotating the current thread's name
during potentially long-running operations such as refresh/load. With
this change, jstack output now includes useful thread names like:
During startup:
"main [invalidating metadata - 128/428 dbs complete]"
While loading a fresh table:
"pool-4-thread-12 [Loading metadata for: foo_db.foo_table] [Loading
metadata for all partition(s) of foo_db.foo_table]"
Pool refreshing metadata for a particular path:
"pool-23-thread-5 [Refreshing file metadata for path:
hdfs://nameservice1/path/to/partdir..."
This patch is tricky to automate tests for, but I verified it manually
by jstacking a catalogd while performing some workload.
Change-Id: Ic7c850d6bb2eedc375ee567c19eb17add335f60c
---
M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
M fe/src/main/java/org/apache/impala/catalog/TableLoader.java
A fe/src/main/java/org/apache/impala/util/ThreadNameAnnotator.java
4 files changed, 99 insertions(+), 19 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/28/11228/1
--
To view, visit http://gerrit.cloudera.org:8080/11228
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic7c850d6bb2eedc375ee567c19eb17add335f60c
Gerrit-Change-Number: 11228
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>
