[GitHub] activemq-artemis pull request #2448: ARTEMIS-2189 allow deleting temp dest w...

2018-12-03 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2448#discussion_r238422596 --- Diff: tests/jms-tests/src/test/java/org/apache/activemq/artemis/jms/tests/TemporaryDestinationTest.java --- @@ -265,6 +266,126 @@ public voi

[GitHub] activemq-artemis pull request #2448: ARTEMIS-2189 allow deleting temp dest w...

2018-12-03 Thread tabish121
Github user tabish121 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2448#discussion_r238416943 --- Diff: tests/jms-tests/src/test/java/org/apache/activemq/artemis/jms/tests/TemporaryDestinationTest.java --- @@ -265,6 +266,126 @@ public vo

[GitHub] activemq-artemis pull request #2448: ARTEMIS-2189 allow deleting temp dest w...

2018-12-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2448#discussion_r238413260 --- Diff: tests/jms-tests/src/test/java/org/apache/activemq/artemis/jms/tests/TemporaryDestinationTest.java --- @@ -265,6 +266,126 @@

[GitHub] activemq-artemis pull request #2448: ARTEMIS-2189 allow deleting temp dest w...

2018-12-03 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/2448 ARTEMIS-2189 allow deleting temp dest when session is closed You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artem