[GitHub] incubator-metron pull request #472: METRON-700: Add hadoop container to metr...

2017-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/472 --- 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

[GitHub] incubator-metron issue #472: METRON-700: Add hadoop container to metron-dock...

2017-03-28 Thread kylerichardson
Github user kylerichardson commented on the issue: https://github.com/apache/incubator-metron/pull/472 Thanks! Next time the storm Dockerfile needs a tweak, I'll rework it to use the packages approach too. --- If your project is set up for it, you can reply to this email and have

Re: [DISCUSS][PROPOSAL] Maven Plugin to build packages with dependencies

2017-03-28 Thread zeo...@gmail.com
I'm game for that On Tue, Mar 28, 2017, 1:19 PM Matt Foley wrote: > Makes sense. > > > > From: Otto Fowler > Date: Tuesday, March 28, 2017 at 10:13 AM > To: "dev@metron.incubator.apache.org" , > Matt Foley

Re: [DISCUSS][PROPOSAL] Maven Plugin to build packages with dependencies

2017-03-28 Thread Matt Foley
Makes sense. From: Otto Fowler Date: Tuesday, March 28, 2017 at 10:13 AM To: "dev@metron.incubator.apache.org" , Matt Foley Subject: Re: [DISCUSS][PROPOSAL] Maven Plugin to build packages with dependencies I am

Re: [DISCUSS][PROPOSAL] Maven Plugin to build packages with dependencies

2017-03-28 Thread Otto Fowler
I am going to proceed on with 2. It is prudent to stand review and criticism before involving infrastructure. On March 28, 2017 at 13:00:46, Matt Foley (ma...@apache.org) wrote: I support option 1 if Infra will do it. Otherwise 2. But it might be easier to achieve the Infra request after we

Re: [DISCUSS][PROPOSAL] Maven Plugin to build packages with dependencies

2017-03-28 Thread Matt Foley
I support option 1 if Infra will do it. Otherwise 2. But it might be easier to achieve the Infra request after we have exited the incubator. --Matt On 3/27/17, 6:25 PM, "zeo...@gmail.com" wrote: I don't have a strong opinion here, but I'm interested to see what the

[GitHub] incubator-metron pull request #494: METRON-806 set posix

2017-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/494 --- 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

[GitHub] incubator-metron issue #494: METRON-806 set posix

2017-03-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/494 Ok - removed from child poms, put in parent --- 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] incubator-metron issue #494: METRON-806 set posix

2017-03-28 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/494 Since all assemblies require this, would it make sense to configure it in pluginManagement in the top level POM? --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-metron pull request #494: METRON-806 set posix

2017-03-28 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/incubator-metron/pull/494 METRON-806 set posix for metron-rest ## Contributor Comments Any new assemblies added to the project need to have the tarLongFileMode set Tested by running on a

[GitHub] incubator-metron issue #472: METRON-700: Add hadoop container to metron-dock...

2017-03-28 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/472 Looks good. I like the packages approach instead of listing out individual modules. +1 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-metron issue #491: METRON-773: Intermittent unit test errors in th...

2017-03-28 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/incubator-metron/pull/491 Ran it several times. Looks good. +1 --- 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