[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 > If the queue and topic with the same name/address were independent, then clients explicitly selecting one or the other would not see each others messages. Messages from a sender with

[GitHub] activemq-artemis pull request #2397: ARTEMIS-2116 Extend tests on producer C...

2018-10-26 Thread feuillemorte
GitHub user feuillemorte opened a pull request: https://github.com/apache/activemq-artemis/pull/2397 ARTEMIS-2116 Extend tests on producer CLI command with message body Added tests on producer cli command Cases: * Produce 10 messages with default message body *

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread grs
Github user grs commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 Yes, I am sure iof all these points. ---

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread grs
Github user grs commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 If the queue and topic with the same name/address were independent, then clients explicitly selecting one or the other would not see each others messages. Messages from a sender with

[GitHub] activemq-artemis issue #2399: ARTEMIS-2150 Counts the number of delivering m...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/2399 @ArthurFritz LGTM, thanks for fixing up the commit message. Just waiting on PR build if good ill look to merge. Thanks for the contribution. ---

[GitHub] activemq-artemis pull request #2398: [ARTEMIS-2150] Counts the number of del...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2398#discussion_r228623226 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/QueueControlImpl.java --- @@ -740,6 +733,56

[GitHub] activemq-artemis pull request #2388: ARTEMIS-1856 support delays before dele...

2018-10-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/2388 ---

[GitHub] activemq-artemis pull request #2399: ARTEMIS-2150 Counts the number of deliv...

2018-10-26 Thread ArthurFritz
GitHub user ArthurFritz opened a pull request: https://github.com/apache/activemq-artemis/pull/2399 ARTEMIS-2150 Counts the number of delivering messages in this queue Create function this counts the number of delivering messages in this queue matching the specified filter.

[GitHub] activemq-artemis pull request #2398: ARTEMIS-2150 Counts the number of deliv...

2018-10-26 Thread ArthurFritz
Github user ArthurFritz closed the pull request at: https://github.com/apache/activemq-artemis/pull/2398 ---

[GitHub] activemq-artemis issue #2398: ARTEMIS-2150 Counts the number of delivering m...

2018-10-26 Thread ArthurFritz
Github user ArthurFritz commented on the issue: https://github.com/apache/activemq-artemis/pull/2398 Okay, i created new Merge Request, adjusted commit message :smiley: ---

[GitHub] activemq-artemis pull request #2398: [ARTEMIS-2150] Counts the number of del...

2018-10-26 Thread ArthurFritz
GitHub user ArthurFritz opened a pull request: https://github.com/apache/activemq-artemis/pull/2398 [ARTEMIS-2150] Counts the number of delivering messages in this queue Create function this counts the number of delivering messages in this queue matching the specified filter.

[GitHub] activemq-artemis pull request #2398: [ARTEMIS-2150] Counts the number of del...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2398#discussion_r228619277 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/QueueControlImpl.java --- @@ -740,6 +733,56

[GitHub] activemq-artemis issue #2388: ARTEMIS-1856 support delays before deleting ad...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/2388 @jbertram it wasn't a biggie, i guess more important things are a foot, so merging. And can always add the extra config to the other as and when needed in future. ---

[GitHub] activemq-artemis issue #2399: ARTEMIS-2150 Counts the number of delivering m...

2018-10-26 Thread ArthurFritz
Github user ArthurFritz commented on the issue: https://github.com/apache/activemq-artemis/pull/2399 @michaelandrepearce Nice, Tks ---

[GitHub] activemq-artemis issue #2398: [ARTEMIS-2150] Counts the number of delivering...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/2398 @ArthurFritz LGTM (when build passes for checkstyle etc etc) could you merge the commits? Also could you tidy the commit message to be in the form: "ARTEMIS- You

[GitHub] activemq-artemis pull request #2399: ARTEMIS-2150 Counts the number of deliv...

2018-10-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/2399 ---

Re: Website

2018-10-26 Thread Bruce Snyder
+1 Bruce On Tue, Oct 23, 2018 at 2:26 PM Clebert Suconic wrote: > What about this. WE could : > > - commit this on git > - upload a snapshot for review under our web-site. > On Mon, Oct 22, 2018 at 8:46 AM Clebert Suconic > wrote: > > > > +1000. We had that options discussed here a few times.

[GitHub] activemq-artemis issue #2396: ARTEMIS-2149 Protecting message.sendBuffer fro...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/2396 Another thing ive noticed is currently where validBuffer is changed its not protected its only volatile, so you could enter syncronized method do the validBuffer check, but

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread grs
Github user grs commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 I am talking only about the AMQP support. It is inconsistent. It does not as you say 'honour the sender'. It effectively forces the behaviour of anycast on all subsequent clients, unless

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 Disagree AMQP is just a protocol. A broker is just another client. There is nothing to say an address cannot be both anycast and multicast. And as such if you have

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 There are two things at play here. AMQP raw protocol and extensions to AMQP that are broker specific. With AMQP raw, there is no such thing as a routing type (this is a broker

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread grs
Github user grs commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 > With AMQP raw, there is no such thing as a routing type (this is a broker concept). Therefore messages sent via AMQP to an address should be routed to any routing type currently configured

[GitHub] activemq-artemis issue #2392: ARTEMIS-2100 address routing-type overridden o...

2018-10-26 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2392 > > With AMQP raw, there is no such thing as a routing type (this is a broker concept). Therefore messages sent via AMQP to an address should be routed to any routing type currently