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 2015/08/31 18:34:59 UTC

activemq-artemis git commit: Fix thread leak in test TransactionContextTest This leak happens with amq5.12.0. It didn't happen with amq5.11.1. Adding connection.start() can work around this seemingly 5.12.0 issue.

Repository: activemq-artemis
Updated Branches:
  refs/heads/master cc9cacef1 -> 1642a856c


Fix thread leak in test TransactionContextTest
  This leak happens with amq5.12.0. It didn't happen
  with amq5.11.1. Adding connection.start() can work
  around this seemingly 5.12.0 issue.


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

Branch: refs/heads/master
Commit: 1642a856c3cdca7ec964d340a660e0c07e38b693
Parents: cc9cace
Author: Howard Gao <ho...@gmail.com>
Authored: Tue Sep 1 00:17:36 2015 +0800
Committer: Howard Gao <ho...@gmail.com>
Committed: Tue Sep 1 00:17:36 2015 +0800

----------------------------------------------------------------------
 .../tests/integration/openwire/amq/TransactionContextTest.java      | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/1642a856/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/openwire/amq/TransactionContextTest.java
----------------------------------------------------------------------
diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/openwire/amq/TransactionContextTest.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/openwire/amq/TransactionContextTest.java
index 2cfcf96..a96fb79 100644
--- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/openwire/amq/TransactionContextTest.java
+++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/openwire/amq/TransactionContextTest.java
@@ -35,6 +35,7 @@ public class TransactionContextTest extends BasicOpenWireTest {
 
    @Before
    public void setup() throws Exception {
+      connection.start();
       underTest = new TransactionContext(connection);
    }