Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19458 )
Change subject: IMPALA-11865: Set thread names for Java thread pools ...................................................................... IMPALA-11865: Set thread names for Java thread pools Currently, some Java thread pools use the default thread names, which take the form "pool-N-thread-M". This sets the thread names to more human-friendly names by adding a ThreadFactory to Executor thread pools in our Java code. This should make the jstack output easier to understand. For example, "pool-8-thread-1" becomes "TableLoadingSubmitterThread-0". Testing: - Ran jstack locally and verified Change-Id: I583093eeb2e92a208ee5ea5a023686e36436c4dc Reviewed-on: http://gerrit.cloudera.org:8080/19458 Reviewed-by: Michael Smith <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/org/apache/impala/catalog/TableLoadingMgr.java M fe/src/main/java/org/apache/impala/catalog/metastore/CatalogMetastoreServer.java M fe/src/main/java/org/apache/impala/hooks/QueryEventHookManager.java M fe/src/main/java/org/apache/impala/service/Frontend.java 5 files changed, 27 insertions(+), 9 deletions(-) Approvals: Michael Smith: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/19458 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I583093eeb2e92a208ee5ea5a023686e36436c4dc Gerrit-Change-Number: 19458 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
