Issue #4121 QueuedThreadPool

ThreadFactory semantic Signed-off-by: Greg Wilkins <gregw@webtide.com>
jetty · Oct 2, 2019 · 7b306d7 · 7b306d7
1 parent c37a4ff
commit 7b306d7
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/jetty-util/src/main/java/org/eclipse/jetty/util/thread/QueuedThreadPool.java b/jetty-util/src/main/java/org/eclipse/jetty/util/thread/QueuedThreadPool.java
@@ -636,9 +636,6 @@ protected void startThread()
         try
         {
             Thread thread = newThread(_runnable);
-            thread.setDaemon(isDaemon());
-            thread.setPriority(getThreadsPriority());
-            thread.setName(_name + "-" + thread.getId());
             if (LOG.isDebugEnabled())
                 LOG.debug("Starting {}", thread);
             _threads.add(thread);
@@ -670,7 +667,11 @@ private boolean addCounts(int deltaThreads, int deltaIdle)
 
     protected Thread newThread(Runnable runnable)
     {
-        return new Thread(_threadGroup, runnable);
+        Thread thread = new Thread(_threadGroup, runnable);
+        thread.setDaemon(isDaemon());
+        thread.setPriority(getThreadsPriority());
+        thread.setName(_name + "-" + thread.getId());
+        return thread;
     }
 
     protected void removeThread(Thread thread)