New ActiveMQ Artemis Release

2016-05-19 Thread Claus Ibsen
Hi Isn't it time to cut a new ActiveMQ Artemis release? The previous 1.2.0 release is 5 months ago. And since the 1.0 release 1 year ago, there has only be 1.1 and 1.2 releases. -- Claus Ibsen - http://davsclaus.com @davsclaus Camel in Action 2: https://www.manning.com/ibsen2

[GitHub] activemq-artemis pull request: Fix Issue #245

2016-05-19 Thread exabrial
Github user exabrial closed the pull request at: https://github.com/apache/activemq-artemis/pull/496 --- 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 fea

Re: Adding support for password masking to the Artemis

2016-05-19 Thread Justin Bertram
If you send a PR then signing a CLA isn't necessary. See here [1] for more details on that. Justin [1] http://mail-archives.apache.org/mod_mbox/incubator-tinkerpop-dev/201502.mbox/%3cd0fe82f6.44d33%25rve...@dotnetrdf.org%3E - Original Message - From: "aclayton" To: dev@activemq.apac

[GitHub] activemq-artemis pull request: [ARTEMIS-506] PagingTest#testDelete...

2016-05-19 Thread clebertsuconic
Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/495#issuecomment-220460874 @gaol Can you review my latest changes on paging, and this can probably be closed? --- If your project is set up for it, you can reply to this email a

[GitHub] activemq-artemis pull request: [ARTEMIS-506] PagingTest#testDelete...

2016-05-19 Thread clebertsuconic
Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/495#issuecomment-220460780 This is exactly the issue I was working against 👍 I wish I had read your PR more properly.. I actually did but I didn't realize it was the

[GitHub] activemq-artemis pull request: fixing typo in doc link

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/527 --- 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: fixing typo in doc link

2016-05-19 Thread clebertsuconic
Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/527#issuecomment-220459920 thanks 👍 --- 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 ha

[GitHub] activemq-artemis pull request: new logging.properties and trace fo...

2016-05-19 Thread clebertsuconic
Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/523#issuecomment-220458254 @mtaylor/@jbertram I have added the thread to the current format with minimal changes... and I have a format commented out on traces that I wil

[GitHub] activemq-artemis pull request: fixing typo in doc link

2016-05-19 Thread pgfox
GitHub user pgfox opened a pull request: https://github.com/apache/activemq-artemis/pull/527 fixing typo in doc link You can merge this pull request into a Git repository by running: $ git pull https://github.com/pgfox/activemq-artemis doc_link_update Alternatively you can re

Re: Adding support for password masking to the Artemis

2016-05-19 Thread aclayton
Will you guys require me to sign a CLA? -- View this message in context: http://activemq.2283324.n4.nabble.com/Adding-support-for-password-masking-to-the-Artemis-tp4711531p4712181.html Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.

[GitHub] activemq-artemis pull request: ARTEMIS-521 - OSGi support - add un...

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/526 --- 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: ARTEMIS-527 - [Artemis Testsuite] T...

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/525 --- 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: Fix Issue #245

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/496#issuecomment-220354749 @exabrial Actually doesn't look like I have permissions to close this PR. Could you please close it. Thanks --- If your project is set up for it, y

[GitHub] activemq-artemis pull request: Fix Issue #245

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/496#issuecomment-220353546 @exabrial I am going to close this PR based on our discussion. We can reopen later if need be. --- If your project is set up for it, you can reply to this em

[GitHub] activemq-artemis pull request: ARTEMIS-521 - OSGi support - add un...

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/526#issuecomment-220353158 @dejanb Looks like you fixed the issue. Could you squash the commits pls. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] activemq-artemis pull request: ARTEMIS-521 - OSGi support - add un...

2016-05-19 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/526#issuecomment-220329595 Apparently org.apache.activemq.artemis.karaf.ArtemisFeatureTest.test failed. Can you look into this, @dejanb? --- If your project is set up for it, you can

[GitHub] activemq-artemis pull request: Fix Issue #245

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/496#issuecomment-220307610 @exabrial I'd be happy to help. I can talk to you on IRC #activemq or on the dev list. --- If your project is set up for it, you can reply to this email and

[GitHub] activemq-artemis pull request: ARTEMIS-521 - OSGi support - add un...

2016-05-19 Thread dejanb
GitHub user dejanb opened a pull request: https://github.com/apache/activemq-artemis/pull/526 ARTEMIS-521 - OSGi support - add unit test You can merge this pull request into a Git repository by running: $ git pull https://github.com/dejanb/activemq-artemis master Alternativel

[GitHub] activemq-artemis pull request: Fix Issue #245

2016-05-19 Thread exabrial
Github user exabrial commented on the pull request: https://github.com/apache/activemq-artemis/pull/496#issuecomment-220302866 I'm not completely sure it's correct actually... a better way would be go make the connection class an interface, rather than a final class. Also a

[GitHub] activemq-artemis pull request: new logging.properties and trace fo...

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/523#issuecomment-220274192 +1 on changing just the trace log format (if possible). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] activemq-artemis pull request: [ARTEMIS-506] PagingTest#testDelete...

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/495#issuecomment-220273890 @clebertsuconic any update on this? --- 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 pro

[GitHub] activemq-artemis pull request: Fix Issue #245

2016-05-19 Thread mtaylor
Github user mtaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/496#issuecomment-220273790 @exabrial Hi, any chance you could update the commit message and we'll get this merged. Thanks --- If your project is set up for it, you can reply to

Re: [Artemis] Why having two different "artemis" scripts

2016-05-19 Thread Martyn Taylor
Hi Lionel, The scripts are slightly different as they do slightly different things. The first script is for generating an Artemis instance, which sets up a new broker in a separate directory, "artemis create broker" The other is for running the broker "artemis run", which takes slight different a

[GitHub] activemq-artemis pull request: ARTEMIS-527 - [Artemis Testsuite] T...

2016-05-19 Thread dudaerich
GitHub user dudaerich opened a pull request: https://github.com/apache/activemq-artemis/pull/525 ARTEMIS-527 - [Artemis Testsuite] TopicControlTest fails You can merge this pull request into a Git repository by running: $ git pull https://github.com/dudaerich/activemq-artemis

[Artemis] Why having two different "artemis" scripts

2016-05-19 Thread Lionel Cons
The Artemis sources contain two slightly different "artemis" scripts (see the diff below). Why not merging them and using only one script? Cheers, Lionel --- artemis-distribution/src/main/resources/bin/artemis 2016-05-02 07:21:26.0 +0200 +++ artemis-cli/src/main/resources/org/apache/a