Re: [VOTE] Apache Artemis 1.3.0

2016-06-13 Thread Hiram Chirino
+1
On Mon, Jun 13, 2016 at 7:40 AM Timothy Bish  wrote:

> +1
>
> * Validated signatures and sums
> * Built from source and ran tests
> * Ran broker from binary archive and ran a few client tests against it
> * Tried out a few of the examples
> * Reviewed some of the docs and README files.
> * Reviewed the license and notice files.
>
> On 06/09/2016 09:03 AM, Martyn Taylor wrote:
> > Hello all,
> >
> > I'd like to propose an Apache Artemis 1.3.0 release.
> >
> > Since 1.2.0 we've had a lot of fixes and improvements:
> >
> > * The OpenWire protocol implementation is now feature complete.
> > * Equivalent implementations of all ActiveMQ 5.x JAAS modules are now
> > available.
> > * An initial implementation of JDBC journal and JDBC storage for Large
> > Message store has been added.
> >- Current support for PostGres, MySQL and Derby
> > * Lots of improvements to the replication HA policy.
> > * Improvements to performance.
> > * Many more bug fixes.
> >
> > The release notes can be found here:
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328978
> >
> > The binary distributions can be found here:
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> >
> >
> > The source archives can be found here:
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> >
> >
> > The Maven repository is here:
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/
> >
> > The source tag:
> >
> https://git-wip-us.apache.org/repos/asf?p=activemq-artemis.git;a=tag;h=refs/tags/1.3.0
> >
> >
> > The project website for that version has been staged to:
> > http://people.apache.org/~martyntaylor/
> >
> > The vote will remain open for 72 hours.
> >
> > [ ] +1 approve the release as Apache Artemis 1.3.0
> > [ ] +0 no opinion
> > [ ] -1 disapprove (and reason why)
> >
> > Here's my (non-binding) +1
> >
> > Regards
> > Martyn
> >
>
>
> --
> Tim Bish
> twitter: @tabish121
> blog: http://timbish.blogspot.com/
>
>


[GitHub] activemq-artemis issue #579: Remove redundant type arguments, add sevntu-che...

2016-06-13 Thread scop
Github user scop commented on the issue:

https://github.com/apache/activemq-artemis/pull/579
  
The last commit is unrelated to the diamond operator, but depends on the 
sevntu addition so I thought I'd just bundle it here.


---
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 #579: Remove redundant type arguments, add sev...

2016-06-13 Thread scop
GitHub user scop opened a pull request:

https://github.com/apache/activemq-artemis/pull/579

Remove redundant type arguments, add sevntu-checkstyle check for it

Hopefully with this check I don't need to keep submitting the redundant 
type args PR's any more :)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/scop/activemq-artemis redundant2

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

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


commit 04fefcc851f153f648f2e13a58b65df4a94a87b9
Author: Ville Skyttä 
Date:   2016-06-13T21:18:58Z

Add Sevntu-checkstyle diamond operator check

commit 051bb174e40aba5d4c5b7539fc36afbf334691f7
Author: Ville Skyttä 
Date:   2016-06-13T21:20:31Z

Remove redundant type arguments




---
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 #578: Update maven-checkstyle-plugin to 2.17, ...

2016-06-13 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/activemq-artemis/pull/578


---
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 issue #578: Update maven-checkstyle-plugin to 2.17, fix vio...

2016-06-13 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/578
  
(I have done a comment earlier here that I removed.. please ignore me.. I 
got confused about what this was about)


---
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 issue #578: Update maven-checkstyle-plugin to 2.17, fix vio...

2016-06-13 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/578
  
I would squash these commits and push -f on your branch
no need to have the checkstyles showing as committs after merge.


I could do it during merge though if you can't 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 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 issue #578: Update maven-checkstyle-plugin to 2.17, fix vio...

2016-06-13 Thread scop
Github user scop commented on the issue:

https://github.com/apache/activemq-artemis/pull/578
  
Done, plus a few similar ones in examples.


---
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 issue #578: Update maven-checkstyle-plugin to 2.17, fix vio...

2016-06-13 Thread mtaylor
Github user mtaylor commented on the issue:

https://github.com/apache/activemq-artemis/pull/578
  
@scop You have 1 checkstyle violation.  Could you please update.  Thanks.

[ERROR] 
src/test/java/org/apache/activemq/artemis/tests/extras/byteman/HierarchicalObjectRepositoryTest.java:[76,7]
 (modifier) RedundantModifier: Redundant 'public' modifier.



---
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 #578: Update maven-checkstyle-plugin to 2.17, ...

2016-06-13 Thread scop
GitHub user scop opened a pull request:

https://github.com/apache/activemq-artemis/pull/578

Update maven-checkstyle-plugin to 2.17, fix violations



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/scop/activemq-artemis checkstyle217

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

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


commit fb56cc7414592777d4e0f3d2ff0b7a5e3a592213
Author: Ville Skyttä 
Date:   2016-06-13T13:46:41Z

Update maven-checkstyle-plugin to 2.17

commit e4937480405d46a28ae36b4167c1680dd4a62386
Author: Ville Skyttä 
Date:   2016-06-13T15:21:43Z

Fix checkstyle curly brace violations

commit d0d023b26259a2c365b15a89ebbdf668d4ff87e1
Author: Ville Skyttä 
Date:   2016-06-13T15:22:33Z

Fix checkstyle whitespace violation

commit 676d2c0916ecce5db5adb92f2fd4e7c952de32b9
Author: Ville Skyttä 
Date:   2016-06-13T15:49:01Z

Fix checkstyle redundant modifier violations




---
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: [VOTE] Apache Artemis 1.3.0

2016-06-13 Thread Timothy Bish

+1

* Validated signatures and sums
* Built from source and ran tests
* Ran broker from binary archive and ran a few client tests against it
* Tried out a few of the examples
* Reviewed some of the docs and README files.
* Reviewed the license and notice files.

On 06/09/2016 09:03 AM, Martyn Taylor wrote:

Hello all,

I'd like to propose an Apache Artemis 1.3.0 release.

Since 1.2.0 we've had a lot of fixes and improvements:

* The OpenWire protocol implementation is now feature complete.
* Equivalent implementations of all ActiveMQ 5.x JAAS modules are now
available.
* An initial implementation of JDBC journal and JDBC storage for Large
Message store has been added.
   - Current support for PostGres, MySQL and Derby
* Lots of improvements to the replication HA policy.
* Improvements to performance.
* Many more bug fixes.

The release notes can be found here:
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328978

The binary distributions can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/


The source archives can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/


The Maven repository is here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1095/

The source tag:
https://git-wip-us.apache.org/repos/asf?p=activemq-artemis.git;a=tag;h=refs/tags/1.3.0


The project website for that version has been staged to:
http://people.apache.org/~martyntaylor/

The vote will remain open for 72 hours.

[ ] +1 approve the release as Apache Artemis 1.3.0
[ ] +0 no opinion
[ ] -1 disapprove (and reason why)

Here's my (non-binding) +1

Regards
Martyn




--
Tim Bish
twitter: @tabish121
blog: http://timbish.blogspot.com/



Re: [VOTE] Apache Artemis 1.3.0

2016-06-13 Thread Clebert Suconic
+1

On Monday, June 13, 2016, Andy Taylor  wrote:

> +1
>
> On 13 June 2016 at 12:58, Christopher Shannon <
> christopher.l.shan...@gmail.com > wrote:
>
> > +1 (binding)
> >
> > On Fri, Jun 10, 2016 at 11:14 AM, Fabio Gomes dos Santos <
> > supergr...@gmail.com > wrote:
> >
> > > +1
> > >
> > > 2016-06-10 3:18 GMT-03:00 Claus Ibsen  >:
> > >
> > > > +1
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > On Thu, Jun 9, 2016 at 3:03 PM, Martyn Taylor  >
> > > wrote:
> > > > > Hello all,
> > > > >
> > > > > I'd like to propose an Apache Artemis 1.3.0 release.
> > > > >
> > > > > Since 1.2.0 we've had a lot of fixes and improvements:
> > > > >
> > > > > * The OpenWire protocol implementation is now feature complete.
> > > > > * Equivalent implementations of all ActiveMQ 5.x JAAS modules are
> now
> > > > > available.
> > > > > * An initial implementation of JDBC journal and JDBC storage for
> > Large
> > > > > Message store has been added.
> > > > >   - Current support for PostGres, MySQL and Derby
> > > > > * Lots of improvements to the replication HA policy.
> > > > > * Improvements to performance.
> > > > > * Many more bug fixes.
> > > > >
> > > > > The release notes can be found here:
> > > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328978
> > > > >
> > > > > The binary distributions can be found here:
> > > > >
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> > > > >
> > > > >
> > > > > The source archives can be found here:
> > > > >
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> > > > >
> > > > >
> > > > > The Maven repository is here:
> > > > >
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/
> > > > >
> > > > > The source tag:
> > > > >
> > > >
> > >
> >
> https://git-wip-us.apache.org/repos/asf?p=activemq-artemis.git;a=tag;h=refs/tags/1.3.0
> > > > >
> > > > >
> > > > > The project website for that version has been staged to:
> > > > > http://people.apache.org/~martyntaylor/
> > > > >
> > > > > The vote will remain open for 72 hours.
> > > > >
> > > > > [ ] +1 approve the release as Apache Artemis 1.3.0
> > > > > [ ] +0 no opinion
> > > > > [ ] -1 disapprove (and reason why)
> > > > >
> > > > > Here's my (non-binding) +1
> > > > >
> > > > > Regards
> > > > > Martyn
> > > >
> > > >
> > > >
> > > > --
> > > > Claus Ibsen
> > > > -
> > > > http://davsclaus.com @davsclaus
> > > > Camel in Action 2: https://www.manning.com/ibsen2
> > > >
> > >
> > >
> > >
> > > --
> > > Fábio Santos
> > > supergr...@gmail.com 
> > > 
> > >
> >
>


-- 
Clebert Suconic


Re: [VOTE] Apache Artemis 1.3.0

2016-06-13 Thread Andy Taylor
+1

On 13 June 2016 at 12:58, Christopher Shannon <
christopher.l.shan...@gmail.com> wrote:

> +1 (binding)
>
> On Fri, Jun 10, 2016 at 11:14 AM, Fabio Gomes dos Santos <
> supergr...@gmail.com> wrote:
>
> > +1
> >
> > 2016-06-10 3:18 GMT-03:00 Claus Ibsen :
> >
> > > +1
> > >
> > >
> > >
> > >
> > >
> > >
> > > On Thu, Jun 9, 2016 at 3:03 PM, Martyn Taylor 
> > wrote:
> > > > Hello all,
> > > >
> > > > I'd like to propose an Apache Artemis 1.3.0 release.
> > > >
> > > > Since 1.2.0 we've had a lot of fixes and improvements:
> > > >
> > > > * The OpenWire protocol implementation is now feature complete.
> > > > * Equivalent implementations of all ActiveMQ 5.x JAAS modules are now
> > > > available.
> > > > * An initial implementation of JDBC journal and JDBC storage for
> Large
> > > > Message store has been added.
> > > >   - Current support for PostGres, MySQL and Derby
> > > > * Lots of improvements to the replication HA policy.
> > > > * Improvements to performance.
> > > > * Many more bug fixes.
> > > >
> > > > The release notes can be found here:
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328978
> > > >
> > > > The binary distributions can be found here:
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> > > >
> > > >
> > > > The source archives can be found here:
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> > > >
> > > >
> > > > The Maven repository is here:
> > > >
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/
> > > >
> > > > The source tag:
> > > >
> > >
> >
> https://git-wip-us.apache.org/repos/asf?p=activemq-artemis.git;a=tag;h=refs/tags/1.3.0
> > > >
> > > >
> > > > The project website for that version has been staged to:
> > > > http://people.apache.org/~martyntaylor/
> > > >
> > > > The vote will remain open for 72 hours.
> > > >
> > > > [ ] +1 approve the release as Apache Artemis 1.3.0
> > > > [ ] +0 no opinion
> > > > [ ] -1 disapprove (and reason why)
> > > >
> > > > Here's my (non-binding) +1
> > > >
> > > > Regards
> > > > Martyn
> > >
> > >
> > >
> > > --
> > > Claus Ibsen
> > > -
> > > http://davsclaus.com @davsclaus
> > > Camel in Action 2: https://www.manning.com/ibsen2
> > >
> >
> >
> >
> > --
> > Fábio Santos
> > supergr...@gmail.com
> > 
> >
>


Re: [VOTE] Apache Artemis 1.3.0

2016-06-13 Thread Christopher Shannon
+1 (binding)

On Fri, Jun 10, 2016 at 11:14 AM, Fabio Gomes dos Santos <
supergr...@gmail.com> wrote:

> +1
>
> 2016-06-10 3:18 GMT-03:00 Claus Ibsen :
>
> > +1
> >
> >
> >
> >
> >
> >
> > On Thu, Jun 9, 2016 at 3:03 PM, Martyn Taylor 
> wrote:
> > > Hello all,
> > >
> > > I'd like to propose an Apache Artemis 1.3.0 release.
> > >
> > > Since 1.2.0 we've had a lot of fixes and improvements:
> > >
> > > * The OpenWire protocol implementation is now feature complete.
> > > * Equivalent implementations of all ActiveMQ 5.x JAAS modules are now
> > > available.
> > > * An initial implementation of JDBC journal and JDBC storage for Large
> > > Message store has been added.
> > >   - Current support for PostGres, MySQL and Derby
> > > * Lots of improvements to the replication HA policy.
> > > * Improvements to performance.
> > > * Many more bug fixes.
> > >
> > > The release notes can be found here:
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328978
> > >
> > > The binary distributions can be found here:
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> > >
> > >
> > > The source archives can be found here:
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/org/apache/activemq/apache-artemis/1.3.0/
> > >
> > >
> > > The Maven repository is here:
> > >
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1095/
> > >
> > > The source tag:
> > >
> >
> https://git-wip-us.apache.org/repos/asf?p=activemq-artemis.git;a=tag;h=refs/tags/1.3.0
> > >
> > >
> > > The project website for that version has been staged to:
> > > http://people.apache.org/~martyntaylor/
> > >
> > > The vote will remain open for 72 hours.
> > >
> > > [ ] +1 approve the release as Apache Artemis 1.3.0
> > > [ ] +0 no opinion
> > > [ ] -1 disapprove (and reason why)
> > >
> > > Here's my (non-binding) +1
> > >
> > > Regards
> > > Martyn
> >
> >
> >
> > --
> > Claus Ibsen
> > -
> > http://davsclaus.com @davsclaus
> > Camel in Action 2: https://www.manning.com/ibsen2
> >
>
>
>
> --
> Fábio Santos
> supergr...@gmail.com
> 
>