You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by el...@apache.org on 2021/01/01 00:21:17 UTC

[hbase] branch master updated: HBASE-25279 Make ZKWatcher ExecutorService launch daemon threads

This is an automated email from the ASF dual-hosted git repository.

elserj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new ec63cc3  HBASE-25279 Make ZKWatcher ExecutorService launch daemon threads
ec63cc3 is described below

commit ec63cc3144d923e83e9fcd7a35d54ec80d4782aa
Author: Josh Elser <el...@apache.org>
AuthorDate: Thu Nov 12 16:52:58 2020 -0500

    HBASE-25279 Make ZKWatcher ExecutorService launch daemon threads
    
    Closes #2651
    
    Signed-off-by: Duo Zhang <zh...@apache.org>
    Signed-off-by: Yulin Niu <ni...@apache.org>
---
 .../src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java b/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java
index 136134d..7a9fdd6 100644
--- a/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java
+++ b/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java
@@ -97,7 +97,7 @@ public class ZKWatcher implements Watcher, Abortable, Closeable {
   // It is ok to do it in a single thread because the Zookeeper ClientCnxn already serializes the
   // requests using a single while loop and hence there is no performance degradation.
   private final ExecutorService zkEventProcessor = Executors.newSingleThreadExecutor(
-    new ThreadFactoryBuilder().setNameFormat("zk-event-processor-pool-%d")
+    new ThreadFactoryBuilder().setNameFormat("zk-event-processor-pool-%d").setDaemon(true)
       .setUncaughtExceptionHandler(Threads.LOGGING_EXCEPTION_HANDLER).build());
 
   private final Configuration conf;