[GitHub] incubator-metron pull request: METRON-132 Monit Integration

2016-05-12 Thread nickwallen
Github user nickwallen commented on the pull request: https://github.com/apache/incubator-metron/pull/118#issuecomment-218927812 @dlyle65535 Recent commits ensure that MySQL is left running when installed so that the `mysql_client` role will work. I then changed the 'start' functiona

Re: [GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread Casey Stella
Cool thanks for the quick turnaround! On Thu, May 12, 2016 at 19:58 mmiklavc wrote: > Github user mmiklavc commented on the pull request: > > > https://github.com/apache/incubator-metron/pull/119#issuecomment-218918935 > > Ok, made a number of changes for review. Thanks for the feedback! > Sh

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on the pull request: https://github.com/apache/incubator-metron/pull/119#issuecomment-218918935 Ok, made a number of changes for review. Thanks for the feedback! Should have the remaining fixes/improvements avail soon and will also test per comments on f

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63115005 --- Diff: metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/filter/query/QueryPcapFilter.java --- @@ -0,0 +1,74 @@ +/** + *

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63114735 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63114632 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63113925 --- Diff: metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/filter/query/QueryPcapFilter.java --- @@ -0,0 +1,74 @@ +/** + *

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63112571 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63112187 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63111447 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63111019 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: Metron 142 Simplify Parser configur...

2016-05-12 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/incubator-metron/pull/120 Metron 142 Simplify Parser configuration This pull request includes several changes related to optimizing configuration for parser topologies. Currently the process for adding parsers

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63105345 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63105055 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/119#issuecomment-218894825 On the whole, this is great. Definitely a great feature and an impressive 2nd PR. Thanks for the contribution! Please make sure you didn't inadverte

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63102275 --- Diff: metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/mr/PcapJob.java --- @@ -33,36 +33,39 @@ import org.apache.hadoop.m

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63102232 --- Diff: metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/mr/PcapJob.java --- @@ -33,36 +33,39 @@ import org.apache.hadoop.m

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63101688 --- Diff: metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/filter/query/QueryPcapFilter.java --- @@ -0,0 +1,74 @@ +/** + *

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63100294 --- Diff: metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/filter/PcapFilters.java --- @@ -0,0 +1,41 @@ +/** + * Licensed

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63099913 --- Diff: metron-platform/metron-common/src/test/java/org/apache/metron/common/query/QueryParserTest.java --- @@ -67,6 +67,7 @@ public void testSi

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63099841 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/query/PredicateProcessor.java --- @@ -18,12 +18,20 @@ pa

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63099540 --- Diff: metron-platform/metron-api/src/main/java/org/apache/metron/pcapservice/PcapReceiverImplRestEasy.java --- @@ -97,6 +97,66 @@ private stat

Re: [DISCUSS] New Incubator Website

2016-05-12 Thread George Vetticaden
+ 1 Looks fantastic! Would love to see these pushed out sooner than later.. There are some content we can include in it..including videos of Meetup, links to various artifacts on the wiki, etc.. -- George Vetticaden Principal, Senior Product Manager for Cyber Security / Metron gvettica...@hortonw

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/119#discussion_r63087348 --- Diff: metron-platform/metron-api/src/main/java/org/apache/metron/pcapservice/PcapReceiverImplRestEasy.java --- @@ -97,6 +97,66 @@ private stat

[GitHub] incubator-metron pull request: METRON-155 Added query filtering ca...

2016-05-12 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/incubator-metron/pull/119 METRON-155 Added query filtering capability for PCAP via Metron REST API https://issues.apache.org/jira/browse/METRON-155 Have not run the Vagrant deploy yet, but this is big PR s

[GitHub] incubator-metron pull request: METRON-151: Remove yum update from ...

2016-05-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/117 --- 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 featu

[GitHub] incubator-metron pull request: METRON-151: Remove yum update from ...

2016-05-12 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/117#issuecomment-218843462 Definitely +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 th

[GitHub] incubator-metron pull request: Metron 146 topology workers

2016-05-12 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/116#issuecomment-218777995 Ah, ok, so this is failing the integration tests because you're not setting the topology.workers property. You can either set it * In [this](https://gith

[GitHub] incubator-metron pull request: [METRON-150] Adding WebSphere parse...

2016-05-12 Thread cestella
Github user cestella commented on the pull request: https://github.com/apache/incubator-metron/pull/115#issuecomment-218772767 Great Job! Have you tested this on the full-dev-vagrant at all? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-metron pull request: [METRON-150] Adding WebSphere parse...

2016-05-12 Thread merrimanr
Github user merrimanr commented on the pull request: https://github.com/apache/incubator-metron/pull/115#issuecomment-218770707 Looks good to me. Nice job @DomenicPuzio. +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-metron pull request: METRON-132 Monit Integration

2016-05-12 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/118#discussion_r62993429 --- Diff: metron-deployment/roles/monit/templates/monit/monit.conf --- @@ -0,0 +1,29 @@ +# +# Licensed to the Apache Software Foundation

Re: Metron UI Meeting Agenda

2016-05-12 Thread Houshang Livian
My apologies Taylor and thank you for the mentoring. Note: This is a weekly re-occurring meeting. I will get the agenda out sooner next week. Houshang Livian Senior User Experience Designer Hortonworks www.hortonworks.com Mobile: (83