You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@activemq.apache.org by Skiler <gi...@git.apache.org> on 2018/02/13 15:58:09 UTC

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

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 <ra...@...>
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

----


---

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

Posted by clebertsuconic <gi...@git.apache.org>.

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


---

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

Posted by Skiler <gi...@git.apache.org>.

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


---

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

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---