[GitHub] activemq-artemis pull request: Update vendor to ASF

GitHub user davsclaus opened a pull request: https://github.com/apache/activemq-artemis/pull/10 Update vendor to ASF You can merge this pull request into a Git repository by running: $ git pull https://github.com/davsclaus/activemq-artemis irc Alternatively you can review and

Re: Website & Dist for Artemis 1.0.0 is up

Hi Should Artemis not be listed as a sub-project from the main ActiveMQ web site at http://activemq.apache.org/ On Fri, May 29, 2015 at 7:36 PM, Hiram Chirino wrote: > The website is up now at: > http://activemq.apache.org/artemis/ > > If anyone needs to tweak it, just check it out and update i

Jenkins build is unstable: ActiveMQ-SysTest-Trunk #11

See

Jenkins build is still unstable: ActiveMQ-SysTest-Trunk » ActiveMQ :: System Tests #11

See

Build failed in Jenkins: ActiveMQ Protocol Buffer #16

See -- Started by an SCM change Building remotely on ubuntu-6 (docker Ubuntu ubuntu) in workspace Unpacking http://archive.apa

[GitHub] activemq pull request: AMQ-5713 - making accessors public to remov...

Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/106 --- 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 en

[GitHub] activemq pull request: AMQ-5713 - making accessors public to remov...

GitHub user mfrazier63 opened a pull request: https://github.com/apache/activemq/pull/106 AMQ-5713 - making accessors public to remove startup error. Includes unit test DestinationBridgeAccessorsTest which replicated the problem and now passes. You can merge this pull request into

[GitHub] activemq pull request: AMQ-5713 - adding getDoHandlReplyTo to sati...

Github user mfrazier63 closed the pull request at: https://github.com/apache/activemq/pull/90 --- 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

Re: Completing the IP clearance for Artemis

Mmmm it wasn't particularly clear that your email was the standard 72 hour lazy consensus vote we usually do for IP Clearance. John On Fri, May 29, 2015 at 1:28 PM Hiram Chirino wrote: > Ok I've not heard any objections from the incubator folks either. > I'll take care of pushing the releases u

Website & Dist for Artemis 1.0.0 is up

The website is up now at: http://activemq.apache.org/artemis/ If anyone needs to tweak it, just check it out and update it via SVN from: https://svn.apache.org/repos/infra/websites/production/activemq/content/artemis The distro has also been deployed to: http://apache.org/dist/activemq/activemq-a

Re: Completing the IP clearance for Artemis

Ok I've not heard any objections from the incubator folks either. I'll take care of pushing the releases up. On Fri, May 29, 2015 at 7:35 AM, Gary Tully wrote: > looks good to me too. > > On 27 May 2015 at 18:04, Hiram Chirino wrote: >> Ok I think the IP clearance form is now complete: >> http:/

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/8#issuecomment-106879458 Please correct those as we have error-prone on the compilation. Look at my last commit and the post to the activemq-dev-list --- If your project is se

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

Github user gaohoward commented on the pull request: https://github.com/apache/activemq-artemis/pull/8#issuecomment-106868119 last time I ran the test suite I got a little over 30% pass rate. So there still a long way to fix it. I don't think we should enable it by default for now.

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

Github user gaohoward commented on the pull request: https://github.com/apache/activemq-artemis/pull/8#issuecomment-106867647 Hi Clebert, those @deprecated tags are from activemq 5 source code, we don't need to correct them imo. This module just get them from amq repository ve

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/8#issuecomment-106826140 Howard, your PR doesn't build because of these issues: /work/apache/six-merge/artemis-selector/target/generated-sources/javacc/org/apache/activemq

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/8#issuecomment-106826266 Please, rebase and fix those? http://errorprone.info/bugpattern/DepAnn --- If your project is set up for it, you can reply to this email and have

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

Github user andytaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/8#issuecomment-106816951 most of the tests fail clebert so we shouldn't have it enabled by default i dont think. --- If your project is set up for it, you can reply to this email and

[GitHub] activemq-artemis pull request: Cleanup issues reported by error pr...

Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/7 --- 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

[GitHub] activemq-artemis pull request: Cleanup issues reported by error pr...

Github user andytaylor commented on the pull request: https://github.com/apache/activemq-artemis/pull/7#issuecomment-106815858 i'll get this merged --- 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

[GitHub] activemq-artemis pull request: ARTEMIS-128 ClassCastException in o...

Github user gaohoward commented on the pull request: https://github.com/apache/activemq-artemis/pull/9#issuecomment-106797988 cherry-picked into ARTEMIS-127 --- 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

[GitHub] activemq-artemis pull request: ARTEMIS-128 ClassCastException in o...

Github user gaohoward closed the pull request at: https://github.com/apache/activemq-artemis/pull/9 --- 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 feat

[GitHub] activemq-artemis pull request: ARTEMIS-128 ClassCastException in o...

Github user gaohoward commented on the pull request: https://github.com/apache/activemq-artemis/pull/9#issuecomment-106797031 ok. --- 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

[GitHub] activemq-artemis pull request: ARTEMIS-128 ClassCastException in o...

Github user clebertsuconic commented on the pull request: https://github.com/apache/activemq-artemis/pull/9#issuecomment-106791273 This PR seems dependent on #8, why don't you cherry-pick this commit into that branch and have a single PR? --- If your project is set up for it, you can

[GitHub] activemq-artemis pull request: ARTEMIS-128 ClassCastException in o...

GitHub user gaohoward opened a pull request: https://github.com/apache/activemq-artemis/pull/9 ARTEMIS-128 ClassCastException in openwire message conversion You can merge this pull request into a Git repository by running: $ git pull https://github.com/gaohoward/activemq-artem

Re: [DISCUSSION] Error prone plugin on codebase

unsubscribe 2015-05-29 4:33 GMT+02:00 Clebert Suconic : > Thiago Kronig has recently added error prone to the Artemis codebase, > so any mistakes would be noticed at compilation time. > > I actually liked it quite a lot... Liking even more now as I realize > what is doing :) > > > > I'm now clean

Re: Completing the IP clearance for Artemis

looks good to me too. On 27 May 2015 at 18:04, Hiram Chirino wrote: > Ok I think the IP clearance form is now complete: > http://svn.apache.org/repos/asf/incubator/public/trunk/content/ip-clearance/hornetq.xml > > Could some more PMC members please review and let me know if it looks > good to the

Re: [DISCUSSION] Error prone plugin on codebase

+1 for adding Error Prone and for adding those 4 mistakes. It should probably be enabled by default since it is compile time and it will help catch problems early. On Fri, May 29, 2015 at 1:47 AM, Thiago Kronig wrote: > It can be very helpful* > > On Fri, May 29, 2015, 02:46 Thiago Kronig wrote

[GitHub] activemq-artemis pull request: ARTEMIS-127 Adding activemq unit te...

GitHub user gaohoward opened a pull request: https://github.com/apache/activemq-artemis/pull/8 ARTEMIS-127 Adding activemq unit test module to Artemis This test module brings in activemq unit tests and run them against Artemis broker. You can merge this pull request into a Git r