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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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.apache.org
Sent: Thursday, May 19, 2016 3:52:21 PM
Subject: Re: Adding support for password masking to the Artemis

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-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 and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 will usually switch to 
(and which I encourage people to use when over-using traces).


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/527.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 #527


commit 4c1e8745cc6d158afad6c7cfb4208e8baeaa85ab
Author: Pat Fox 
Date:   2016-05-19T21:27:43Z

fixing typo in doc link




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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, 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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/526.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 #526


commit 159aa6353b04a8741244719fd6cbea150165b94e
Author: Dejan Bosanac 
Date:   2016-05-19T11:56:00Z

ARTEMIS-521 - OSGi support - add unit test




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 at Devoxx UK in two weeks I'm leading a workshop with the goal is to
get this working 100%. If any of you will be available, or present
physically, that'd be awesome!
On May 19, 2016 5:30 AM, "Martyn Taylor"  wrote:

> @exabrial  Hi, any chance you could update
> the commit message and we'll get this merged.
>
> Thanks
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> 

>



---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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 arguments /
configuration.

You're right, this could be refactored into a single script to do both
things (I can see some repetition in the code).  What you've included here
seems reasonable, we'd need to test it of course.  Could you please open a
pull request and we'll test it and get it merged should everything be OK.

Thanks a lot
Martyn


[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-527

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/525.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 #525


commit 17322fb57a38633edb2f754ffb97bbd2dd08bed8
Author: Erich Duda 
Date:   2016-05-18T13:54:48Z

ARTEMIS-527 - [Artemis Testsuite] TopicControlTest fails




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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/activemq/artemis/cli/commands/bin/artemis
 2016-05-02 07:21:26.0 +0200
@@ -16,7 +16,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
-if [ -z "$ARTEMIS_HOME" ] ; then
+if [ -z "$ARTEMIS_INSTANCE" ] ; then
 
   ## resolve links - $0 may be a link to ActiveMQ's home
   PRG="$0"
@@ -37,15 +37,21 @@
 fi
   done
 
-  ARTEMIS_HOME=`dirname "$PRG"`
+  ARTEMIS_INSTANCE=`dirname "$PRG"`
   cd "$saveddir"
 
   # make it fully qualified
-  ARTEMIS_HOME=`cd "$ARTEMIS_HOME/.." && pwd`
+  ARTEMIS_INSTANCE=`cd "$ARTEMIS_INSTANCE/.." && pwd`
 fi
 
 # Set Defaults Properties
-JAVA_ARGS="-XX:+UseParallelGC -XX:+AggressiveOpts -XX:+UseFastAccessorMethods 
-Xms512M -Xmx1024M"
+ARTEMIS_LOGGING_CONF="file:$ARTEMIS_INSTANCE/etc/logging.properties"
+ARTEMIS_DATA_DIR="$ARTEMIS_INSTANCE/data"
+ARTEMIS_LOG_MANAGER=org.jboss.logmanager.LogManager
+
+# Load Profile Data
+. "$ARTEMIS_INSTANCE/etc/artemis.profile"
+
 CLASSPATH="$ARTEMIS_HOME/lib/artemis-boot.jar"
 
 # OS specific support.
@@ -65,8 +71,8 @@
 
 # For Cygwin, ensure paths are in UNIX format before anything is touched
 if $cygwin ; then
-  [ -n "$ARTEMIS_HOME" ] &&
-ARTEMIS_HOME=`cygpath --unix "$ARTEMIS_HOME"`
+  [ -n "$ARTEMIS_INSTANCE" ] &&
+ARTEMIS_INSTANCE=`cygpath --unix "$ARTEMIS_INSTANCE"`
   [ -n "$JAVA_HOME" ] &&
 JAVA_HOME=`cygpath --unix "$JAVA_HOME"`
 fi
@@ -93,16 +99,24 @@
   exit 1
 fi
 
-
 if $cygwin ; then
   JAVA_HOME=`cygpath --windows "$JAVA_HOME"`
   ARTEMIS_HOME=`cygpath --windows "$ARTEMIS_HOME"`
   CLASSPATH=`cygpath --windows "$CLASSPATH"`
 fi
 
-exec "$JAVACMD" $JAVA_ARGS $ARTEMIS_CLUSTER_PROPS \
+exec "$JAVACMD" ${java-opts} \
+$JAVA_ARGS \
+-Xbootclasspath/a:"$ARTEMIS_HOME/lib/${logmanager}" \
+-Djava.security.auth.login.config="$ARTEMIS_INSTANCE/etc/login.config" \
+$ARTEMIS_CLUSTER_PROPS \
 -classpath "$CLASSPATH" \
 -Dartemis.home="$ARTEMIS_HOME" \
+-Dartemis.instance="$ARTEMIS_INSTANCE" \
 
-Djava.library.path="$ARTEMIS_HOME/bin/lib/linux-i686:$ARTEMIS_HOME/bin/lib/linux-x86_64"
 \
+-Djava.io.tmpdir="$ARTEMIS_INSTANCE/tmp" \
+-Ddata.dir="$ARTEMIS_DATA_DIR" \
+-Djava.util.logging.manager="$ARTEMIS_LOG_MANAGER" \
+-Dlogging.configuration="$ARTEMIS_LOGGING_CONF" \
 $DEBUG_ARGS \
 org.apache.activemq.artemis.boot.Artemis "$@"