[GitHub] metron issue #814: METRON-1277 Add match statement to Stellar language

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/814 the issue with MAP() is resolved as well ---

[GitHub] metron issue #814: METRON-1277 Add match statement to Stellar language

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/814 OK, the validation issues are a big big deal. I have changed the match functionality so that they work in that context, supporting guard expressions and requiring the default operation.

[GitHub] metron pull request #803: Metron-1252: Build ui for grouping alerts into met...

2017-10-30 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/803#discussion_r147853082 --- Diff: metron-interface/metron-alerts/e2e/alerts-list/tree-view/tree-view.po.ts --- @@ -159,4 +159,34 @@ export class TreeViewPage { return

[GitHub] metron pull request #803: Metron-1252: Build ui for grouping alerts into met...

2017-10-30 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/803#discussion_r147853051 --- Diff: metron-interface/metron-alerts/e2e/alerts-list/tree-view/tree-view.po.ts --- @@ -159,4 +159,34 @@ export class TreeViewPage { return

[GitHub] metron pull request #803: Metron-1252: Build ui for grouping alerts into met...

2017-10-30 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/803#discussion_r147853067 --- Diff: metron-interface/metron-alerts/e2e/alerts-list/tree-view/tree-view.po.ts --- @@ -159,4 +159,34 @@ export class TreeViewPage { return

[GitHub] metron issue #803: Metron-1252: Build ui for grouping alerts into meta alert...

2017-10-30 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/803 I am consistently getting these e2e test failures: ``` 1) meta-alerts workflow should have all the steps for meta alerts workflow - Error: Timeout - Async callback was not invoked

[GitHub] metron issue #820: METRON-1287 Full Dev Fails When Installing EPEL Repositor...

2017-10-30 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/820 I was able to successfully build full dev with this change. +1 ---

Re: [DISCUSS] Using Yarn package manager for metron-alerts

2017-10-30 Thread Michael Miklavcic
Would love to revive this - I think this could help drastically reduce our build times for metron-interface, which locally just took me 9 minutes in non-parallel mode with -DskipTests set. This is a really good suggestion even just for the offline install and version locking, as pointed out by

[GitHub] metron pull request #819: METRON-1267: Alerts UI returns a 404 when refreshi...

2017-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/819 ---

[GitHub] metron pull request #821: METRON-1281: Remove hard-coded indices from the Al...

2017-10-30 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/821 METRON-1281: Remove hard-coded indices from the Alerts UI ## Contributor Comments This PR adjusts the Alerts UI and REST app to remove the need to hard-code the indices in the Alerts UI

[GitHub] metron pull request #814: METRON-1277 Add match statement to Stellar languag...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/814#discussion_r147785631 --- Diff: metron-stellar/stellar-common/src/main/antlr4/org/apache/metron/stellar/common/generated/Stellar.g4 --- @@ -272,4 +278,30 @@

[GitHub] metron pull request #814: METRON-1277 Add match statement to Stellar languag...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/814#discussion_r147784210 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/StellarCompiler.java --- @@ -105,59 +107,77 @@ public Object

[GitHub] metron pull request #820: METRON-1287 Full Dev Fails When Installing EPEL Re...

2017-10-30 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/820 METRON-1287 Full Dev Fails When Installing EPEL Repository This fixes a change in the EPEL repository that is causing deployments of Full Dev to fail. ``` TASK [epel : Get epel-repo

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/817 ---

[GitHub] metron issue #817: METRON-1283: Install Elasticsearch template as a part of ...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/817 +1, thanks! ---

[GitHub] metron pull request #801: METRON-1254: Conditionals as map keys do not funct...

2017-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/801 ---

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147737598 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -86,8 +86,13 @@ rm ${SENSOR}.template The stock set of Elasticsearch templates

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147725937 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -86,8 +86,13 @@ rm ${SENSOR}.template The stock set of Elasticsearch

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147714561 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147711822 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147708307 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147702906 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147699187 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #819: METRON-1267: Alerts UI returns a 404 when refreshi...

2017-10-30 Thread iraghumitra
GitHub user iraghumitra opened a pull request: https://github.com/apache/metron/pull/819 METRON-1267: Alerts UI returns a 404 when refreshing the alerts-list page Fixed the 404 error when refreshing alerts-ui(added the missing base-dir path). Looks like the last fix didn't

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147686518 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread anandsubbu
Github user anandsubbu commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147681110 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT

[GitHub] metron pull request #817: METRON-1283: Install Elasticsearch template as a p...

2017-10-30 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/817#discussion_r147670198 --- Diff: metron-platform/metron-elasticsearch/README.md --- @@ -81,3 +81,13 @@ curl -XPUT