Re: [DISCUSS] Modify bylaws to allow speculative branches

2017-04-24 Thread Justin Leet
I'd prefer to still keep "Review then Commit" but I'd be interested in lowering the review bar for it as long as the branch gets a solid final review before merge. I don't think a feature branch needs to be micromanaged, but people should still take a quick glance at the commits that come

Re: [DISCUSS] Modify bylaws to allow speculative branches

2017-04-24 Thread Michael Miklavcic
"Branch committers ... do not cast binding votes or vetoes in the project." It sounds similar to what we do for other PRs except in this case it's 1..n contributors instead of just one. The step for getting the feature branch merged into trunk sounds reasonable and clear enough, but as Casey

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Casey Stella
haha you're not thick; you're noticing a bit of a hack that should probably be corrected. The use of parsers here is entirely to opportunistically take advantage of the fact taht the parsers have as direct dependencies all of the stellar functions in Metron. This was done prior to us being able

Quick Dev - Atlas Images

2017-04-24 Thread Nick Allen
Right now, we have the images that get pushed to Atlas for Quick Dev versioned independently from the rest of Metron. We currently have versions 0.1.0 and 0.2.0. What happens when a user downloads an official release of Metron, like 0.3.1, and

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112972432 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apache

[GitHub] incubator-metron pull request #538: METRON-868 Fix documentation on building...

2017-04-24 Thread justinleet
Github user justinleet closed the pull request at: https://github.com/apache/incubator-metron/pull/538 --- 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 pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112972023 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy Apache

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112968303 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -13,7 +13,9 @@ Getting Started The computer used to deploy

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/538 Would this be fixed if we just ran build-rpms every time? In other words got rid of the profile flag requirement? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-metron issue #544: METRON-866 Quickdev and Fulldev readme misses D...

2017-04-24 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/544 Done. Please take a look. --- 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] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 +1 - pulled in the dependency changes, deleted my .m2 cache and the build ran as expected. Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112961765 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112961515 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @mmiklavc Added a quick note, and listed those dependencies off. Let me know if there's anything else you want in there. --- If your project is set up for it, you can reply to this

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/538 @justinleet Works for me. Please indicate that we have a dependency across projects that is not shown via the dependency mechanism. Specifically, we have a dependency on the tarballs

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @mmiklavc I made the proposed change and pushed it so it can at least be seen. I'll can easily adjust if you either don't want to do this or have suggestions or anything else. --- If

Re: metron UI

2017-04-24 Thread Justin Leet
To elaborate on Ryan's reply a bit, the UI is a fairly new component in Metron and was merged into master not long ago. It's definitely open to being iterated and improved on, and getting feedback on direction users would like it to go would be a great contribution in and of itself (especially as

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Otto Fowler
OK, not to be thick, I just don’t see where any class from metron-parsers gets called or instantiated. ¯\_(ツ)_/¯ On April 24, 2017 at 09:55:38, Casey Stella (ceste...@gmail.com) wrote: All of the current stellar functions are depended on either directly or indirectly by the parser. On Mon, Apr

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112953201 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

Re: metron UI

2017-04-24 Thread Ryan Merriman
What features would you like to see included? On Mon, Apr 24, 2017 at 8:51 AM, moshe jarusalem wrote: > Hi All, > I have recently run metron UI. I am a bit surprised because it has very few > features such as configuring some ingestions and topologies. > > Have I not

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Casey Stella
All of the current stellar functions are depended on either directly or indirectly by the parser. On Mon, Apr 24, 2017 at 9:54 AM, Otto Fowler wrote: > too bad there isn’t a pr out for that kind of stuff ;) > > There are no stellar functions in parsers, except what is

metron UI

2017-04-24 Thread moshe jarusalem
Hi All, I have recently run metron UI. I am a bit surprised because it has very few features such as configuring some ingestions and topologies. Have I not configured it properly or its features are limited at this time ? Regards,

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112950226 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable, but

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/538 @mmiklavc @dlyle65535 Do we want to compromise on this for now by removing the dependency, and kick out another ticket to investigate a more robust way to manage this? Mike,

No longer incubating, but newly hatched!

2017-04-24 Thread Casey Stella
Hi All, Some of you know this already and some of you might not, but as of the last ASF board meeting we became a top level project with me serving as the Vice President of Apache Metron. The good people at the ASF press office scheduled some press early this morning. - NASDAQ GlobeNewswire

Re: Silly question about zk_load_configs.sh

2017-04-24 Thread Otto Fowler
So, this script is in common, but: export JAR=metron-parsers-$METRON_VERSION-uber.jar Is the jar it executes. The class it references is : export CLASSNAME=“org.apache.metron.common.cli.ConfigurationManager" So, why is it calling the parsers jar?

[GitHub] incubator-metron issue #220: Metron-342 Fix permissions of {{ metron_directo...

2017-04-24 Thread gspeter
Github user gspeter commented on the issue: https://github.com/apache/incubator-metron/pull/220 @simonellistonball ok. Thank you. --- 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