You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cl...@apache.org on 2017/05/05 18:48:56 UTC

[1/2] activemq-artemis git commit: ARTEMIS-874: ThreadGroup memory leak

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 <de...@gmail.com>
Authored: Fri May 5 22:10:59 2017 +0500
Committer: dOkI <de...@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);


[2/2] activemq-artemis git commit: This closes #1254

Posted by cl...@apache.org.
This closes #1254


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

Branch: refs/heads/1.x
Commit: 5ec545a7928403a59c0372f26648acbf91036e25
Parents: 94f721e e4c4539
Author: Clebert Suconic <cl...@apache.org>
Authored: Fri May 5 14:48:44 2017 -0400
Committer: Clebert Suconic <cl...@apache.org>
Committed: Fri May 5 14:48:44 2017 -0400

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