You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Carsten Ziegeler (JIRA)" <ji...@apache.org> on 2018/03/21 08:35:00 UTC

[jira] [Resolved] (FELIX-5813) EventAdmin async threads should be named

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

Carsten Ziegeler resolved FELIX-5813.
-------------------------------------
    Resolution: Fixed

Thanks for your patch, [~volteanu]

It's applied in rev 1827394

> EventAdmin async threads should be named
> ----------------------------------------
>
>                 Key: FELIX-5813
>                 URL: https://issues.apache.org/jira/browse/FELIX-5813
>             Project: Felix
>          Issue Type: Bug
>          Components: Event Admin
>    Affects Versions: eventadmin-1.4.10
>            Reporter: Valentin Olteanu
>            Assignee: Carsten Ziegeler
>            Priority: Minor
>             Fix For: eventadmin-1.4.12
>
>
> Similar to FELIX-5006, the threads created in the async thread pools, should also be named. 
> The patch should be trivial:
> {code}
> 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;
>                  }
>              };
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)