[ 
https://issues.apache.org/jira/browse/IMPALA-11865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Joe McDonnell resolved IMPALA-11865.
------------------------------------
    Fix Version/s: Impala 4.3.0
       Resolution: Fixed

> Set thread names for Java thread pools
> --------------------------------------
>
>                 Key: IMPALA-11865
>                 URL: https://issues.apache.org/jira/browse/IMPALA-11865
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Frontend
>    Affects Versions: Impala 4.3.0
>            Reporter: Joe McDonnell
>            Assignee: Joe McDonnell
>            Priority: Minor
>             Fix For: Impala 4.3.0
>
>
> When I do a jstack on catalogd, a lot of the thread names are very generic:
> {noformat}
> "pool-9-thread-1" #29 prio=5 os_prio=0 tid=0x000000000d60e000 nid=0x1eb754 
> waiting on condition [0x00007f8364536000]
> {noformat}
> There was already work done on this in IMPALA-7450, which adds information 
> when the threads are processing work. I think we should set the thread name 
> as part of the thread pool definition so that idle threads are easy to 
> identify.
> Here are some locations:
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java#L267]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/catalog/TableLoadingMgr.java#L151]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/catalog/TableLoadingMgr.java#L165]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/catalog/TableLoadingMgr.java#L267]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/catalog/metastore/CatalogMetastoreServer.java#L111]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/hooks/QueryEventHookManager.java#L120]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/service/Frontend.java#L397]
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/service/Frontend.java#L443-L447]
> We do this in other locations by setting a ThreadFactory, for example in 
> CatalogHmsAPIHelper.java:
> {noformat}
>   private static final ExecutorService fallbackFdLoaderPool = Executors
>       .newFixedThreadPool(20,
>           new ThreadFactoryBuilder().setDaemon(true)
>               .setNameFormat("HMS-Filemetadata-loader-%d").build());{noformat}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to