You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by clebertsuconic <gi...@git.apache.org> on 2017/11/29 02:20:23 UTC
[GitHub] activemq-artemis pull request #1678: ARTEMIS-1529 Fixing Ref count over asyn...
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/1678
ARTEMIS-1529 Fixing Ref count over asynchronous ack
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/clebertsuconic/activemq-artemis ARTEMIS-1529
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/1678.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1678
----
commit f989a7ae959b9a61ac3dcfedcae0541ba402508f
Author: Tomas Kratky <tk...@redhat.com>
Date: 2017-11-28T21:37:46Z
ARTEMIS-1529 Adding test on durable topics
commit ee6a0fe6fadf670e4691b2e19e21aaef8b79d92d
Author: Clebert Suconic <cl...@apache.org>
Date: 2017-11-29T02:08:05Z
ARTEMIS-1529 Fixing Ref count over asynchronous ack
----
---
[GitHub] activemq-artemis issue #1678: ARTEMIS-1529 Fixing Ref count over asynchronou...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/1678
please, do not merge this yet
---
[GitHub] activemq-artemis issue #1678: ARTEMIS-1529 Fixing Ref count over asynchronou...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/1678
@tkratky I am adding the test you create through this PR.. attributing the authorship to you.
if you could review please?
---
[GitHub] activemq-artemis pull request #1678: ARTEMIS-1529 Fixing Ref count over asyn...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/1678
---
[GitHub] activemq-artemis issue #1678: ARTEMIS-1529 Fixing Ref count over asynchronou...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/1678
with this fix, yes
---
[GitHub] activemq-artemis issue #1678: ARTEMIS-1529 Fixing Ref count over asynchronou...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/1678
it's fine now.. false alarm
---
[GitHub] activemq-artemis issue #1678: ARTEMIS-1529 Fixing Ref count over asynchronou...
Posted by kornys <gi...@git.apache.org>.
Github user kornys commented on the issue:
https://github.com/apache/activemq-artemis/pull/1678
@clebertsuconic dothese tests pass with current fixes?
---