[GitHub] activemq-artemis issue #724: [ARTEMIS-684] Client connection are not balance...

2016-08-23 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/724 @bayern39 if you actually agree with my change, you can probably close this one. Take a look at #730 and if you agree please close this one. Also please review

[GitHub] activemq-artemis issue #724: [ARTEMIS-684] Client connection are not balance...

2016-08-23 Thread bayern39
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/724 OK,I will do it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wish

[GitHub] activemq-artemis issue #724: [ARTEMIS-684] Client connection are not balance...

2016-08-23 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/724 I have sent another PR: #730 Can you take a look please? Please close this one when that one is merged. I'm not adding your example as I don't see much val

[GitHub] activemq-artemis issue #724: [ARTEMIS-684] Client connection are not balance...

2016-08-17 Thread bayern39
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/724 In fact,I think the no seed Random is better than it has. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project d

[GitHub] activemq-artemis issue #724: [ARTEMIS-684] Client connection are not balance...

2016-08-17 Thread bayern39
Github user bayern39 commented on the issue: https://github.com/apache/activemq-artemis/pull/724 @jbertram Hi Justin,I have try another way fixed it! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does n

[GitHub] activemq-artemis issue #724: [ARTEMIS-684] Client connection are not balance...

2016-08-17 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/724 I'm not sure I agree with this change since it will impact existing load-balancing semantics which users may depend on. Connection load-balancing is done per connection factory. Connecti