[GitHub] activemq-artemis pull request #1869: ARTEMIS-1680 - Message load balancing t...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1869: ARTEMIS-1680 - Message load balancing t...

jbertram-2
GitHub user Skiler opened a pull request:

    https://github.com/apache/activemq-artemis/pull/1869

    ARTEMIS-1680 - Message load balancing type isn't updated in the linke…

    …d bindings when wildcards are used
   
    This makes the linked address consistent  when a message is sent to a linked address and after the wildcard address is subscribed

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Skiler/activemq-artemis wildcard

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/1869.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 #1869
   
----
commit 788f817c5cc7e388ca280f03680ac925d75cd52c
Author: raul.valdoleiros <raul.valdoleiros@...>
Date:   2018-02-13T15:55:46Z

    ARTEMIS-1680 - Message load balancing type isn't updated in the linked bindings when wildcards are used
   
    This makes the linked address consistent  when a message is sent to a linked address and after the wildcard address is subscribed

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis issue #1869: ARTEMIS-1680 - Message load balancing type isn...

jbertram-2
Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1869
 
    can you tidy up the commit message...
   
   
    I'm a bit flexible with long names.. some people are strict with 80 characters on the first line... but your line is way too big.. it's hard to read with a git log


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis issue #1869: ARTEMIS-1680 - Message load balancing type isn...

jbertram-2
In reply to this post by jbertram-2
Github user Skiler commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1869
 
    Hi @clebertsuconic ,
   
    I hope it's good now. Let me know if you have any thing else.
   
    Thanks


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1869: ARTEMIS-1680 - Message load balancing t...

jbertram-2
In reply to this post by jbertram-2
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/1869


---