Problem with MQTT pattern subscription in a cluster

2016-10-14 Thread Antoine Toulme
Hello, we are trying out ActiveMQ Artemis with MQTT. We are interested in using pattern subscription such as foo/# to listen to a set of topics. We created a cluster in which one client publishes to a node, while an other one subscribes to another. When using pattern subscriptions, we see that

[GitHub] activemq-artemis pull request #843: ARTEMIS-800: Add + pattern to the MQTT p...

2016-10-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/843 --- 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 wishes so, or if the featu

[GitHub] activemq-artemis pull request #841: ARTEMIS-795 Tests for temporary destinat...

2016-10-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/841 --- 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 wishes so, or if the featu

[GitHub] activemq-artemis issue #839: ARTEMIS-793 Improvement to OSGi integration

2016-10-14 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/839 @graben could you please squash you please rebase on top of master vs using the merge commit. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] activemq-artemis issue #839: ARTEMIS-793 Improvement to OSGi integration

2016-10-14 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/839 @mtaylor that merge script we always use will fix it. I already tried it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] activemq-artemis issue #839: ARTEMIS-793 Improvement to OSGi integration

2016-10-14 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/839 @clebertsuconic great. It looks like @graben has addressed your comments. If you're happy I'll merge this? --- If your project is set up for it, you can reply to this email and have your

Re: Problem with MQTT pattern subscription in a cluster

2016-10-14 Thread Martyn Taylor
As a quick check can you tell me what message-load-policy you have configured on your cluster connection. Could you try setting this to STRICT? If that doesn't work a reproducer would be the best thing to provide. I can manage to get somewhere with the modified cluster test, send a PR and I'll t

[GitHub] activemq-artemis pull request #845: ARTEMIS-801 Decode URL paths

2016-10-14 Thread dudaerich
GitHub user dudaerich opened a pull request: https://github.com/apache/activemq-artemis/pull/845 ARTEMIS-801 Decode URL paths If the path to file contains some special characters, they are encoded in URL form using % syntax. We should decode such path when it is used as path

[GitHub] activemq-artemis issue #838: ARTEMIS-268 Adds tests that shows issue with pr...

2016-10-14 Thread tabish121
Github user tabish121 commented on the issue: https://github.com/apache/activemq-artemis/pull/838 PR Updated with fix and additional tests to ensure that even under a TX session the message sent to a presettled receiver are consumed. --- If your project is set up for it, you can repl

Re: Problem with MQTT pattern subscription in a cluster

2016-10-14 Thread Antoine Toulme
> On Oct 14, 2016, at 3:01 AM, Martyn Taylor wrote: > > As a quick check can you tell me what message-load-policy you have > configured on your cluster connection. Could you try setting this to > STRICT? It was unset, I tried setting to STRICT. That didn’t change the behavior. > > If that does

[GitHub] activemq-artemis issue #840: ARTEMIS-794 Add tests for handling of various o...

2016-10-14 Thread tabish121
Github user tabish121 commented on the issue: https://github.com/apache/activemq-artemis/pull/840 Updated tests to show that Rejected is not being treated as a terminal outcome and message is redelivered instead of archived in the DLQ --- If your project is set up for it, you can rep

[GitHub] activemq pull request #205: Master

2016-10-14 Thread priyankpande9
GitHub user priyankpande9 opened a pull request: https://github.com/apache/activemq/pull/205 Master Learning You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/activemq master Alternatively you can review and apply these chan

Re: Problem with MQTT pattern subscription in a cluster

2016-10-14 Thread Antoine Toulme
I have a test showing a cluster using MQTT, but it passes. Either the test is bogus, the issue is intermittent, the issue was fixed since 1.4.0 and/or my cluster configuration is wrong. Previously, I have found the issue to be intermittent. Please feel free to take a look at the test. If it loo