[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75949744 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/dsl/functions/SummaryStatisticsFunctions.java --- @@ -0,0 +1,198

[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75949574 --- Diff: metron-platform/metron-common/src/test/java/org/apache/metron/common/stellar/StellarStatisticalFunctionsTest.java --- @@ -0,0 +1,146 @

[GitHub] incubator-metron pull request #227: METRON-388: Stellar - Unable to Operate ...

2016-08-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/227 --- 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 #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75934951 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/dsl/functions/SummaryStatisticsFunctions.java --- @@ -0,0 +1,198 @@

[GitHub] incubator-metron pull request #226: METRON-377 Enable Profiles that Use Non-...

2016-08-23 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/226#discussion_r75933922 --- Diff: metron-platform/metron-common/src/test/java/org/apache/metron/common/stellar/StellarStatisticsFunctionsTest.java --- @@ -0,0 +1,244 @@

[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75934713 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/dsl/functions/SummaryStatisticsFunctions.java --- @@ -0,0 +1,198

[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75933510 --- Diff: metron-platform/metron-common/src/test/java/org/apache/metron/common/stellar/StellarStatisticalFunctionsTest.java --- @@ -0,0 +1,146 @@

[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75933341 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/dsl/functions/SummaryStatisticsFunctions.java --- @@ -0,0 +1,198 @@

[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/225#discussion_r75933182 --- Diff: metron-platform/metron-common/src/main/java/org/apache/metron/common/dsl/functions/SummaryStatisticsFunctions.java --- @@ -0,0 +1,198 @@

[GitHub] incubator-metron issue #227: METRON-388: Stellar - Unable to Operate on Resu...

2016-08-23 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/227 +1 Looks solid --- 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 wish

[GitHub] incubator-metron pull request #227: METRON-388: Stellar - Unable to Operate ...

2016-08-23 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/incubator-metron/pull/227 METRON-388: Stellar - Unable to Operate on Result of If-Then-Else As per [METROn-388](https://issues.apache.org/jira/browse/METRON-388), we added a regression to Stellar around conditiona

[GitHub] incubator-metron pull request #215: METRON-377 Enable Profiles that Use Non-...

2016-08-23 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/incubator-metron/pull/215 --- 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 f

[GitHub] incubator-metron issue #215: METRON-377 Enable Profiles that Use Non-Single ...

2016-08-23 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/215 Replaces by #226 --- 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 w

[GitHub] incubator-metron pull request #226: METRON-377 Enable Profiles that Use Non-...

2016-08-23 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/incubator-metron/pull/226 METRON-377 Enable Profiles that Use Non-Single Pass Summary Functions ### [METRON-377](https://issues.apache.org/jira/browse/METRON-377) This pull request replaces #215 .

[GitHub] incubator-metron pull request #214: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/incubator-metron/pull/214 --- 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 f

[GitHub] incubator-metron issue #214: METRON-372 Enhance Statistical Operations Avail...

2016-08-23 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/214 Closing in favor of #225 --- 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 enabl

[GitHub] incubator-metron pull request #225: METRON-372 Enhance Statistical Operation...

2016-08-23 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/incubator-metron/pull/225 METRON-372 Enhance Statistical Operations Available for Use with the Profiler ### [METRON-372](https://issues.apache.org/jira/browse/METRON-372) This PR serves as a replacement

[GitHub] incubator-metron pull request #216: METRON-341: Create stellar enrichment ad...

2016-08-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/216 --- 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 issue #219: METRON-196 Fix support of ansible 2.1 in metron...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/219 We version lock Ansible because we have seen some issues with regression. This JSON body issue was one of them. FWIW, I'm happy to support 2.1, but we'll need to version lock on 2.1 The

[GitHub] incubator-metron issue #219: METRON-196 Fix support of ansible 2.1 in metron...

2016-08-23 Thread 2xyo
Github user 2xyo commented on the issue: https://github.com/apache/incubator-metron/pull/219 Yes, it breaks support of 2.0.0.2 in order to support the [latest stable version of Ansible](https://www.ansible.com/blog/ansible-2.1). Which one do you prefer? :) --- If your pro

[GitHub] incubator-metron issue #219: METRON-196 Fix support of ansible 2.1 in metron...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/219 So, if I apply this patch, it no longer works in 2.0.0.2, is that correct? --- 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 issue #219: METRON-196 Fix support of ansible 2.1 in metron...

2016-08-23 Thread 2xyo
Github user 2xyo commented on the issue: https://github.com/apache/incubator-metron/pull/219 @dlyle65535 : I downgrade to pip 2.0.0.2 and replay the playbook: it fails at "[Add Elasticsearch templates for topologies](https://gist.githubusercontent.com/2xyo/5e99fa83c8419c8298d03bda0ee0

[GitHub] incubator-metron issue #221: METRON-384 Allow elasticsearch to bind to loopb...

2016-08-23 Thread 2xyo
Github user 2xyo commented on the issue: https://github.com/apache/incubator-metron/pull/221 Thanks @dlyle65535 for the feedback. Would you prefer a configuration like this? Old `full-dev-platform/group_vars/all`: ``` elasticsearch_network_interface: eth0

[GitHub] incubator-metron issue #222: METRON-385 Create Ambari Service Definition for...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/222 +1 looks good! --- 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 wish

[GitHub] incubator-metron pull request #222: METRON-385 Create Ambari Service Definit...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/222#discussion_r75869071 --- Diff: metron-platform/metron-elasticsearch/src/main/config/elasticsearch.properties --- @@ -57,10 +57,13 @@ org.apache.metron.metrics.Teleme

[GitHub] incubator-metron pull request #222: METRON-385 Create Ambari Service Definit...

2016-08-23 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/222#discussion_r75867968 --- Diff: metron-platform/metron-elasticsearch/src/main/config/elasticsearch.properties --- @@ -57,10 +57,13 @@ org.apache.metron.metrics.Teleme

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

2016-08-23 Thread 2xyo
Github user 2xyo commented on the issue: https://github.com/apache/incubator-metron/pull/220 here is my setup: # Metron full dev platform (Windows - 21/08/2016) Two Virtualbox (5.1.41) VM - Client : - Ubuntu (from ubuntu-16.04.1-desktop-amd64.iso)

[GitHub] incubator-metron issue #216: METRON-341: Create stellar enrichment adapter

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/216 +1 - ran this up on Vagrant. Worked well! --- 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 #222: METRON-385 Create Ambari Service Definit...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/222#discussion_r75862174 --- Diff: metron-platform/metron-elasticsearch/src/main/config/elasticsearch.properties --- @@ -57,10 +57,13 @@ org.apache.metron.metrics.Teleme

[GitHub] incubator-metron issue #221: METRON-384 Allow elasticsearch to bind to loopb...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/221 I like what you did here, but could you expose it as a parameter and default it to ipv4 instead? --- If your project is set up for it, you can reply to this email and have your reply ap

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

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/220 I don't see that error when I deploy. Can you tell me a little bit about your environment? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-metron issue #219: METRON-196 Fix support of ansible 2.1 in metron...

2016-08-23 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/219 Thanks for the contribution- did you happen to test this on 2.0.0.2? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-metron pull request #213: METRON-368 Simplify Profile Configuratio...

2016-08-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/213 --- 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 issue #213: METRON-368 Simplify Profile Configuration with ...

2016-08-23 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/213 About to get my merge on. --- 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 enabl

[GitHub] incubator-metron issue #213: METRON-368 Simplify Profile Configuration with ...

2016-08-23 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/213 Yep I took a look last night and just did again. Looks good --- 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] incubator-metron issue #213: METRON-368 Simplify Profile Configuration with ...

2016-08-23 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/213 @cestella Give this one more look before we merge it. I added the additional configuration elements for `HBaseTestingUtility` and also suppressed excessive logging coming out of the `me