[FELIX-5813] EventAdmin async threads should be named

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: eventadmin-1.4.10
• Fix Version/s: eventadmin-1.5.0
• Component/s: Event Admin
• Flags: Patch
• Assignee: Carsten Ziegeler
• Reporter: Valentin Olteanu
• Created: 20/Mar/18 17:26
• Updated: 07/May/18 03:45
• Resolved: 21/Mar/18 08:34

Description

Similar to FELIX-5006, the threads created in the async thread pools, should also be named.

The patch should be trivial:

diff --git a/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/tasks/DefaultThreadPool.java b/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/tasks/DefaultThreadPool.java
index c582fdf47..b71f51bdc 100644
--- a/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/tasks/DefaultThreadPool.java
+++ b/eventadmin/impl/src/main/java/org/apache/felix/eventadmin/impl/tasks/DefaultThreadPool.java
@@ -76,6 +76,7 @@ public class DefaultThreadPool
                     thread.setPriority( Thread.NORM_PRIORITY );
                     thread.setDaemon( true );

+                    thread.setName("EventAdminAsyncThread #" + threadCounter.getAndIncrement());
                     return thread;
                 }
             };