Repository: activemq-artemis
Updated Branches:
  refs/heads/1.x 94f721e88 -> 5ec545a79


ARTEMIS-874: ThreadGroup memory leak


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/e4c4539c
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/e4c4539c
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/e4c4539c

Branch: refs/heads/1.x
Commit: e4c4539c8fefefec23d174441144e1c2a05cf7ca
Parents: 94f721e
Author: dOkI <denis.kirpichen...@gmail.com>
Authored: Fri May 5 22:10:59 2017 +0500
Committer: dOkI <denis.kirpichen...@gmail.com>
Committed: Fri May 5 22:10:59 2017 +0500

----------------------------------------------------------------------
 .../apache/activemq/artemis/utils/ActiveMQThreadFactory.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/e4c4539c/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/ActiveMQThreadFactory.java
----------------------------------------------------------------------
diff --git 
a/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/ActiveMQThreadFactory.java
 
b/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/ActiveMQThreadFactory.java
index e5f76d3..1644715 100644
--- 
a/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/ActiveMQThreadFactory.java
+++ 
b/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/ActiveMQThreadFactory.java
@@ -24,7 +24,7 @@ import java.util.concurrent.atomic.AtomicInteger;
 
 public final class ActiveMQThreadFactory implements ThreadFactory {
 
-   private final ThreadGroup group;
+   private String groupName;
 
    private final AtomicInteger threadCount = new AtomicInteger(0);
 
@@ -59,7 +59,7 @@ public final class ActiveMQThreadFactory implements 
ThreadFactory {
     * @param tccl      the context class loader of newly created threads
     */
    public ActiveMQThreadFactory(final String groupName, String prefix, final 
boolean daemon, final ClassLoader tccl) {
-      group = new ThreadGroup(groupName + "-" + System.identityHashCode(this));
+      this.groupName = groupName;
 
       this.prefix = prefix;
 
@@ -97,7 +97,7 @@ public final class ActiveMQThreadFactory implements 
ThreadFactory {
    }
 
    private Thread createThread(final Runnable command) {
-      final Thread t = new Thread(group, command, prefix + 
threadCount.getAndIncrement() + " (" + group.getName() + ")");
+      final Thread t = new Thread(command, prefix + 
threadCount.getAndIncrement() + " (" + groupName + ")");
       t.setDaemon(daemon);
       t.setPriority(threadPriority);
       t.setContextClassLoader(tccl);

Reply via email to