[GitHub] metron pull request #706: METRON-1119 Bad Out-of-Order Logic for SaltyRowKey...

2017-08-17 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/706 METRON-1119 Bad Out-of-Order Logic for SaltyRowKeyBuilder This is a bug that was previously Identified by @mattf-horton https://github.com/apache/metron/pull/622#pullrequestreview-51351898

[GitHub] metron pull request #705: METRON-1118 Ignore Profile with Bad 'onlyif' or 'f...

2017-08-17 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/705 METRON-1118 Ignore Profile with Bad 'onlyif' or 'foreach' Expressions The Profiler needs to gracefully ignore profiles that have a bad 'onlyif' or 'foreach' expression. Under certain conditions

[GitHub] metron pull request #700: METRON-1077: Use column meta data end point in ale...

2017-08-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/700#discussion_r133840897 --- Diff: metron-interface/metron-alerts/src/app/service/rest-api-impl.ts --- @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software Foundation

Re: [Question] Stopping Storm, Metron & Kafka doesn't stop all Storm processes?

2017-08-17 Thread Laurens Vets
That seems close to the issue that I'm having... But there's a part I don't quite understand in my case. Everything's working fine for days and then suddenly, Java throws an error (see other mail to the mailing list) and I can't get Metron stable again. It's always either the indexingBolt or

Re: REST and ManagementUI : Grok

2017-08-17 Thread Otto Fowler
Should the HDFS Service ever work outside the metron application area in hdfs? Should it be bound? Also, should the UI ‘know’ that it needs to get grok rules from hdfs service or from grok service or should it just ask the grok service and let it take care of things? On August 17, 2017 at

[GitHub] metron issue #699: METRON-1068: Use Elastic end point in alerts ui

2017-08-17 Thread iraghumitra
Github user iraghumitra commented on the issue: https://github.com/apache/metron/pull/699 @nickwallen Is the rest-api available @ http://node1:8082 I tried running the tests on my setup they seemed fine. When the tests run a chrome browser window opens (in minimised mode) can you

[GitHub] metron pull request #703: METRON-1115: Jackson is not properly serializing r...

2017-08-17 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/703 METRON-1115: Jackson is not properly serializing readMetadata and mergeMetadata SensorParserConfig properties ## Contributor Comments The getter methods for readMetadata and mergeMetadata

Re: [Question] Stopping Storm, Metron & Kafka doesn't stop all Storm processes?

2017-08-17 Thread zeo...@gmail.com
I used to run into similar issues when my environment was resource constrained but never ran it to root cause. It has been a long time since I was in this scenario to re-test. https://issues.apache.org/jira/projects/METRON/issues/METRON-485 Jon On Thu, Aug 17, 2017 at 12:49 PM Laurens Vets

[Question] Stopping Storm, Metron & Kafka doesn't stop all Storm processes?

2017-08-17 Thread Laurens Vets
Hi, Simple question, when I stop Metron, Kafka & Storm via Ambari, I still see the storm worker processes running, is this expected?

[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-08-17 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/702 METRON-1114: Add group by capabilities to search REST endpoint ## Contributor Comments This PR adds a group by feature to the search REST endpoint. This PR is fairly complex so I don't think

Re: REST and ManagementUI : Grok

2017-08-17 Thread Otto Fowler
Is there a way to debug the UI and the Rest Service running locally, connected to vagrant node1? On August 17, 2017 at 12:01:49, Ryan Merriman (merrim...@gmail.com) wrote: Check out line 336 in

Re: REST and ManagementUI : Grok

2017-08-17 Thread Ryan Merriman
Check out line 336 in https://github.com/apache/metron/blob/master/metron-interface/metron-config/src/app/sensors/sensor-parser-config/sensor-parser-config.component.ts. That's where the check is done to determine if a grok parser has been selected. I would also check the browser console for

[GitHub] metron pull request #699: METRON-1068: Use Elastic end point in alerts ui

2017-08-17 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/699#discussion_r133756111 --- Diff: metron-interface/metron-alerts/e2e/login/login.e2e-spec.ts --- @@ -0,0 +1,44 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] metron pull request #699: METRON-1068: Use Elastic end point in alerts ui

2017-08-17 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/699#discussion_r133743885 --- Diff: metron-interface/metron-alerts/e2e/login/login.e2e-spec.ts --- @@ -0,0 +1,44 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] metron pull request #694: METRON-1085: Add REST endpoint to save a user prof...

2017-08-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/694#discussion_r133735783 --- Diff: dependencies_with_url.csv --- @@ -112,6 +112,7 @@ com.fasterxml.jackson.core:jackson-core:jar:2.6.6:compile,ASLv2,https://github.c

[GitHub] metron pull request #698: METRON-1111 Mislabeled Configuration Property in A...

2017-08-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/698 --- 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: [DISCUSS] Using Yarn package manager for metron-alerts

2017-08-17 Thread Ryan Merriman
Thanks for this Raghu. You make a pretty compelling argument. I'm +1 on moving to yarn. Ryan On Wed, Aug 16, 2017 at 3:51 PM, Nick Allen wrote: > It is also my understanding that > ​there is no hard cut-over to yarn > . > ​After we > introduce the yarn.lock > ​ > ​,​ > as

[GitHub] metron issue #701: METRON-1113: UpdateControllerIntegrationTest fails with 5...

2017-08-17 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/701 Yep, this makes sense. The static block ensures that the mock hbase table will be created first. +1 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] metron pull request #701: METRON-1113: UpdateControllerIntegrationTest fails...

2017-08-17 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/701 METRON-1113: UpdateControllerIntegrationTest fails with 500 error ## Contributor Comments This PR moves the MockHBaseTableProvider.addToCache call from @BeforeClass methods to a static block

[GitHub] metron pull request #694: METRON-1085: Add REST endpoint to save a user prof...

2017-08-17 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/694#discussion_r133714315 --- Diff: dependencies_with_url.csv --- @@ -112,6 +112,7 @@ com.fasterxml.jackson.core:jackson-core:jar:2.6.6:compile,ASLv2,https://github.c

[GitHub] metron issue #698: METRON-1111 Mislabeled Configuration Property in Ambari

2017-08-17 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/698 +1 by inspection --- 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

[GitHub] metron pull request #699: Metron-1068: Use Elastic end point in alerts ui

2017-08-17 Thread iraghumitra
GitHub user iraghumitra opened a pull request: https://github.com/apache/metron/pull/699 Metron-1068: Use Elastic end point in alerts ui ## Contributor Comments Added support for fetching alerts data using metron-rest instead of fetching from ES. - Added option to