You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by gemmellr <gi...@git.apache.org> on 2018/08/07 15:16:08 UTC
[GitHub] activemq-artemis pull request #2221: ARTEMIS-1978: update to proton-j 0.27.3...
GitHub user gemmellr opened a pull request:
https://github.com/apache/activemq-artemis/pull/2221
ARTEMIS-1978: update to proton-j 0.27.3 to resolve sequencing issues
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gemmellr/activemq-artemis delivery-multiplexing
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/2221.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 #2221
----
commit b496cfc5c9a3d0878568b415d965ec51d049b3c7
Author: Robbie Gemmell <ro...@...>
Date: 2018-08-07T14:35:16Z
ARTEMIS-1978: update to proton-j 0.27.3 to resolve sequencing issues
Adds test exposing broker behaviour from issues stemming from PROTON-1892 and PROTON-1901
----
---
[GitHub] activemq-artemis issue #2221: ARTEMIS-1978: update to proton-j 0.27.3 to res...
Posted by gemmellr <gi...@git.apache.org>.
Github user gemmellr commented on the issue:
https://github.com/apache/activemq-artemis/pull/2221
I'd also like to backport this change to 2.6.x given the severity of the problems, which is why 0.27.3 is used and not yet 0.28.1.
The backport needs a small test change rather than direct cherry pick, so I can raise a separate PR for that once this change is merged.
---
[GitHub] activemq-artemis issue #2221: ARTEMIS-1978: update to proton-j 0.27.3 to res...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2221
@gemmellr I'm a bit out of context on the test change you need to make.. I tried to do it myself but I didn't have the context of your changes. can y ou cherry-pick it then as you mentioned? I won't do it until you cherry-pick it into there.
I have not been using PRs on 2.6.x (they are useles on other branches than master)...
So, please make sure you reference the commit from upstream? commit <ID> will do on your commit message.
---
[GitHub] activemq-artemis pull request #2221: ARTEMIS-1978: update to proton-j 0.27.3...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2221
---