Use named daemon threads for callback processing

mishail · mishail · commit 741492cd1181 · 2016-04-14T13:57:39.000-07:00
diff --git a/src/main/java/com/github/dockerjava/jaxrs/async/AbstractCallbackNotifier.java b/src/main/java/com/github/dockerjava/jaxrs/async/AbstractCallbackNotifier.java
@@ -10,6 +10,7 @@
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
+import java.util.concurrent.ThreadFactory;
 
 import javax.ws.rs.ProcessingException;
 import javax.ws.rs.client.Invocation.Builder;
@@ -18,13 +19,17 @@
 import com.github.dockerjava.api.async.ResultCallback;
 import com.github.dockerjava.core.async.ResponseStreamProcessor;
 import com.github.dockerjava.jaxrs.util.WrappedResponseInputStream;
+import com.google.common.util.concurrent.ThreadFactoryBuilder;
 
 public abstract class AbstractCallbackNotifier<T> implements Callable<Void> {
 
     private final ResponseStreamProcessor<T> responseStreamProcessor;
 
     private final ResultCallback<T> resultCallback;
 
+    private static final ThreadFactory FACTORY =
+            new ThreadFactoryBuilder().setDaemon(true).setNameFormat("dockerjava-jaxrs-async-%d").build();
+
     protected final Builder requestBuilder;
 
     protected AbstractCallbackNotifier(ResponseStreamProcessor<T> responseStreamProcessor,
@@ -76,7 +81,7 @@ public Void call() throws Exception {
 
     public static <T> Future<Void> startAsyncProcessing(AbstractCallbackNotifier<T> callbackNotifier) {
 
-        ExecutorService executorService = Executors.newSingleThreadExecutor();
+        ExecutorService executorService = Executors.newSingleThreadExecutor(FACTORY);
         Future<Void> response = executorService.submit(callbackNotifier);
         executorService.shutdown();
         return response;
