[GitHub] metron pull request #1294: METRON-1930: Update webpack-dev-server in Alerts ...

2018-12-10 Thread sardell
GitHub user sardell opened a pull request: https://github.com/apache/metron/pull/1294 METRON-1930: Update webpack-dev-server in Alerts UI ## Contributor Comments After running `npm audit` in the Alerts UI, it looks like we need to update our version of webpack-dev-server.

[GitHub] metron pull request #1291: METRON-1922: Escaping incorrectly handled in curr...

2018-12-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1291 ---

[GitHub] metron pull request #1293: METRON-1928: Bump Metron version to 0.7.0 for rel...

2018-12-10 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1293 METRON-1928: Bump Metron version to 0.7.0 for release ## Contributor Comments Moving version to 0.7.0 prior to cutting an RC. Reviewers should validate that 0.6.1 is gone (except a

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-09 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r240064005 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-09 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r240063655 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1292: METRON-1925 Provide Verbose View of Profile Resul...

2018-12-09 Thread nickwallen
GitHub user nickwallen reopened a pull request: https://github.com/apache/metron/pull/1292 METRON-1925 Provide Verbose View of Profile Results in REPL ## Motivation When viewing profile measurements in the REPL using PROFILE_GET, you simply get back a list of values. It is

[GitHub] metron pull request #1292: METRON-1925 Provide Verbose View of Profile Resul...

2018-12-09 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/1292 ---

[GitHub] metron pull request #1278: METRON-1892 Parser Debugger Should Load Config Fr...

2018-12-09 Thread nickwallen
GitHub user nickwallen reopened a pull request: https://github.com/apache/metron/pull/1278 METRON-1892 Parser Debugger Should Load Config From Zookeeper When using the parser debugger functions created in #1265, the user has to manually specify the parser configuration. This

[GitHub] metron pull request #1278: METRON-1892 Parser Debugger Should Load Config Fr...

2018-12-09 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/1278 ---

[GitHub] metron pull request #1284: METRON-1867 Remove `/api/v1/update/replace` endpo...

2018-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1284 ---

[GitHub] metron pull request #1289: METRON-1810 Storm Profiler Intermittent Test Fail...

2018-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1289 ---

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-07 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239860797 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-07 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239859491 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-07 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239855809 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-07 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239847486 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-06 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239664781 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-06 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239611104 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-12-06 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r239608145 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1283: METRON-1909: Remove http filter from release util...

2018-12-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1283 ---

[GitHub] metron pull request #1289: METRON-1810 Storm Profiler Intermittent Test Fail...

2018-12-06 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1289#discussion_r239577883 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/ProfilePeriod.java --- @@ -151,6 +152,8 @@ public String

[GitHub] metron pull request #1292: METRON-1925 Provide Verbose View of Profile Resul...

2018-12-06 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/1292 METRON-1925 Provide Verbose View of Profile Results in REPL ## Motivation When viewing profile measurements in the REPL using PROFILE_GET, you simply get back a list of values. It is

[GitHub] metron pull request #1289: METRON-1810 Storm Profiler Intermittent Test Fail...

2018-12-06 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1289#discussion_r239568177 --- Diff: metron-analytics/metron-profiler-common/src/main/java/org/apache/metron/profiler/DefaultMessageDistributor.java --- @@ -315,41 +308,51 @@

[GitHub] metron pull request #:

2018-12-06 Thread JonZeolla
Github user JonZeolla commented on the pull request: https://github.com/apache/metron/commit/ba9ddda6d742a0df96e24a1573f44482020c25fb#commitcomment-31583772 In metron-contrib/metron-docker/compose/kafkazk/Dockerfile: In metron-contrib/metron-docker/compose/kafkazk/Dockerfile on

[GitHub] metron pull request #:

2018-12-05 Thread akju
Github user akju commented on the pull request: https://github.com/apache/metron/commit/ba9ddda6d742a0df96e24a1573f44482020c25fb#commitcomment-31576319 In line 35, 'chmod 755 /opt/kafka/bin/wait-for-it.sh'. File 'wait-for-it.sh' is not in this folder. Should it be

[GitHub] metron pull request #1291: METRON-1922: Escaping incorrectly handled in curr...

2018-12-05 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1291 METRON-1922: Escaping incorrectly handled in current aesh version ## Contributor Comments SPLIT('foo\\bar', '\\') doesn't work properly. It does after bumping the aesh version. Seems like

[GitHub] metron pull request #1290: METRON-1923: change default consumer name

2018-12-05 Thread Condla
GitHub user Condla opened a pull request: https://github.com/apache/metron/pull/1290 METRON-1923: change default consumer name ## Contributor Comments Fixes https://issues.apache.org/jira/browse/METRON-1923 * Default consumer name used sensor topic name. * This

[GitHub] metron pull request #1289: METRON-1810 Storm Profiler Intermittent Test Fail...

2018-12-05 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/1289 METRON-1810 Storm Profiler Intermittent Test Failure This PR hopefully resolves some of the more frequent, intermittent `ProfilerIntegrationTest` failures. ### Testing Run

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-05 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r239114286 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/bulk/BulkDocumentWriter.java --- @@ -0,0 +1,79 @@ +/**

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-05 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r239065568 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchDao.java --- @@ -196,7 +196,7 @@ public

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-05 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r239054925 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriterTest.java --- @@ -18,170 +18,241

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-05 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r239054070 --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/update/Document.java --- @@ -89,46 +91,29 @@ public void

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-04 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r238883436 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchDao.java --- @@ -196,7 +196,7 @@ public

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-04 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r238881060 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriter.java --- @@ -56,90 +60,111 @@

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-04 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r238876879 --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/update/Document.java --- @@ -89,46 +91,29 @@ public void

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-12-04 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r238882021 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriterTest.java --- @@ -18,170 +18,241

[GitHub] metron pull request #1280: METRON-1869 Unable to Sort an Escalated Meta Aler...

2018-12-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1280 ---

[GitHub] metron pull request #1286: METRON-1889: Add any missing timestamp fields to ...

2018-12-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1286 ---

[GitHub] metron pull request #1286: METRON-1889: Add any missing timestamp fields to ...

2018-12-03 Thread mmiklavc
Github user mmiklavc closed the pull request at: https://github.com/apache/metron/pull/1286 ---

[GitHub] metron pull request #1286: METRON-1889: Add any missing timestamp fields to ...

2018-12-03 Thread mmiklavc
GitHub user mmiklavc reopened a pull request: https://github.com/apache/metron/pull/1286 METRON-1889: Add any missing timestamp fields to unified enrichment topology ## Contributor Comments https://issues.apache.org/jira/browse/METRON-1889 This is done in reference

[GitHub] metron pull request #1280: METRON-1869 Unable to Sort an Escalated Meta Aler...

2018-12-03 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/1280 ---

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237921728 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237920926 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237918400 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237921563 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237920415 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237926005 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/SwaggerConfig.java --- @@ -17,22 +17,66 @@ */ package

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237918925 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1281: METRON-1895: Add Knox SSO as an option in Metron

2018-12-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1281#discussion_r237922637 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KnoxSSOAuthenticationFilter.java --- @@ -0,0 +1,314 @@ +/**

[GitHub] metron pull request #1285: METRON-1913: metron-alert UI - Build broken by mi...

2018-12-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1285 ---

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237875330 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237870087 --- Diff: metron-platform/metron-parsers/src/test/resources/config/RegularExpressionsInvalidParserConfig.json --- @@ -0,0 +1,208 @@ +{ +

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237869538 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237869285 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237868794 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237866676 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1269: METRON-1879 Allow Elasticsearch to Auto-Generate ...

2018-11-30 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1269#discussion_r237852978 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriter.java --- @@ -56,90 +58,107 @@

[GitHub] metron pull request #1287: METORN-1915: Disable vagrant hostmanager in the d...

2018-11-29 Thread JonZeolla
GitHub user JonZeolla opened a pull request: https://github.com/apache/metron/pull/1287 METORN-1915: Disable vagrant hostmanager in the development guest ## Contributor Comments This makes node1 resolve to 127.0.0.1 inside of the VM, instead of 192.168.66.121 by not modifying

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237718325 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237716600 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237715657 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1286: METRON-1889: Add any missing timestamp fields to ...

2018-11-29 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/metron/pull/1286 METRON-1889: Add any missing timestamp fields to unified enrichment topology ## Contributor Comments https://issues.apache.org/jira/browse/METRON-1889 This is done in reference

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237715210 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237714079 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237708788 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237707637 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237699908 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread jagdeepsingh2
Github user jagdeepsingh2 commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237698406 --- Diff: metron-platform/metron-parsers/src/test/resources/config/RegularExpressionsInvalidParserConfig.json --- @@ -0,0 +1,208 @@ +{ +

[GitHub] metron pull request #1285: METRON-1913 metron-alert UI - Build broken by mis...

2018-11-29 Thread tiborm
GitHub user tiborm opened a pull request: https://github.com/apache/metron/pull/1285 METRON-1913 metron-alert UI - Build broken by missing transitive dependency ## Contributor Comments Metron Alert UI build broken by a removed transitive dependency. flatmap-steam was

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237651771 --- Diff: metron-platform/metron-parsers/src/test/resources/config/RegularExpressionsInvalidParserConfig.json --- @@ -0,0 +1,208 @@ +{ +

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237650548 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237649967 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237649469 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237648142 --- Diff: metron-platform/metron-parsers/README.md --- @@ -52,6 +52,62 @@ There are two general types types of parsers: This is using the

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237574064 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237584581 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1245: METRON-1795: Initial Commit for Regular Expressio...

2018-11-29 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1245#discussion_r237585161 --- Diff: metron-platform/metron-parsers/src/test/java/org/apache/metron/parsers/regex/RegularExpressionsParserTest.java --- @@ -0,0 +1,152 @@ +/**

[GitHub] metron pull request #1249: METRON-1815: Separate metron-parsers into metron-...

2018-11-29 Thread justinleet
Github user justinleet closed the pull request at: https://github.com/apache/metron/pull/1249 ---

[GitHub] metron pull request #1284: METRON-1867 Remove `/api/v1/update/replace` endpo...

2018-11-29 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/1284 METRON-1867 Remove `/api/v1/update/replace` endpoint The `/api/v1/update/replace` endpoint is no longer used. This is dead code and should be removed. I have tried to isolate the

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1247 ---

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-28 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r237204881 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/components/ElasticSearchComponent.java --- @@

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-28 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r237202358 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchSearchIntegrationTest.java --- @@

[GitHub] metron pull request #1283: METRON-1909: Remove http filter from release util...

2018-11-28 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1283 METRON-1909: Remove http filter from release utils changelog generation ## Contributor Comments Removing the http clause entirely. Can be tested by running the script and making sure

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r236844166 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriter.java --- @@ -56,90 +58,107 @@

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r236843942 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/bulk/ElasticsearchBulkDocumentWriter.java --- @@ -0,0

[GitHub] metron pull request #1276: METRON-1888 Default Topology Settings in MPack Ca...

2018-11-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1276 ---

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236771158 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/components/ElasticSearchComponent.java --- @@

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236767632 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchSearchIntegrationTest.java --- @@

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236765451 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -186,4 +195,114 @@ public

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236765044 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -186,4 +195,114 @@ public

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r236738173 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/bulk/ElasticsearchBulkDocumentWriter.java --- @@ -0,0

[GitHub] metron pull request #1280: METRON-1869 Unable to Sort an Escalated Meta Aler...

2018-11-27 Thread nickwallen
GitHub user nickwallen reopened a pull request: https://github.com/apache/metron/pull/1280 METRON-1869 Unable to Sort an Escalated Meta Alert This fixes a bug that causes meta-alerts to not be visible in the Alerts UI when the UI is sorted by 'alert_status' and a meta-alert has

[GitHub] metron pull request #1280: METRON-1869 Unable to Sort an Escalated Meta Aler...

2018-11-27 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/1280 ---

[GitHub] metron pull request #1282: METRON-1881: Update the bro plugin used in metron...

2018-11-27 Thread JonZeolla
GitHub user JonZeolla opened a pull request: https://github.com/apache/metron/pull/1282 METRON-1881: Update the bro plugin used in metron development ## Contributor Comments Once the metron-bro-plugin-kafka v0.3 is released, this will move full-dev to use it. ##

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236652983 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -186,4 +195,114 @@ public

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236644723 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/components/ElasticSearchComponent.java --- @@

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-27 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236643962 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchSearchIntegrationTest.java --- @@

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-11-26 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r236530989 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/writer/ElasticsearchWriter.java --- @@ -56,90 +58,107 @@

[GitHub] metron pull request #1254: METRON-1849 Elasticsearch Index Write Functionali...

2018-11-26 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1254#discussion_r236530583 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/bulk/ElasticsearchBulkDocumentWriter.java --- @@ -0,0 +1,184

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-26 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236524985 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/ElasticsearchSearchIntegrationTest.java --- @@

[GitHub] metron pull request #1247: METRON-1845 Correct Test Data Load in Elasticsear...

2018-11-26 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236521989 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -186,4 +195,114 @@ public

  1   2   3   4   5   6   7   8   9   10   >