[GitHub] metron pull request #1222: Updated the Readme.md for Regular expressions par...

Github user jagdeepsingh2 closed the pull request at: https://github.com/apache/metron/pull/1222 ---

[GitHub] metron pull request #1222: Updated the Readme.md for Regular expressions par...

GitHub user jagdeepsingh2 opened a pull request: https://github.com/apache/metron/pull/1222 Updated the Readme.md for Regular expressions parser. Configuration field explanation for regular expressions parser. ## Contributor Comments [Please place any comments here. A

[GitHub] metron issue #1222: Updated the Readme.md for Regular expressions parser.

Github user jagdeepsingh2 commented on the issue: https://github.com/apache/metron/pull/1222 This PR needs to be ignored. ---

[jira] [Commented] (METRON-1805) Provide a default value for the Storm topology.max.spout.pending setting

[ https://issues.apache.org/jira/browse/METRON-1805?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637544#comment-16637544 ] ASF GitHub Bot commented on METRON-1805: GitHub user merrimanr opened a pull request:

[GitHub] metron pull request #1221: METRON-1805: Provide a default value for the Stor...

GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/1221 METRON-1805: Provide a default value for the Storm topology.max.spout.pending setting ## Contributor Comments Users have reported problems in the random and batch indexing topologies when

[jira] [Created] (METRON-1805) Provide a default value for the Storm topology.max.spout.pending setting

Ryan Merriman created METRON-1805: - Summary: Provide a default value for the Storm topology.max.spout.pending setting Key: METRON-1805 URL: https://issues.apache.org/jira/browse/METRON-1805 Project:

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637512#comment-16637512 ] ASF GitHub Bot commented on METRON-1681: Github user merrimanr commented on the issue:

[GitHub] metron issue #1213: METRON-1681: Decouple the ParserBolt from the Parse exec...

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1213 I just pushed a commit out based on some feedback from Nick: - The stellar context is now passed into the ParserRunner abstraction as a dependency. I imagine different environments will

[jira] [Commented] (METRON-1771) Update REST endpoints to support eventually consistent UI updates

[ https://issues.apache.org/jira/browse/METRON-1771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637507#comment-16637507 ] ASF GitHub Bot commented on METRON-1771: Github user merrimanr commented on a diff in the pull

[GitHub] metron pull request #1190: METRON-1771: Update REST endpoints to support eve...

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1190#discussion_r222470180 --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/MultiIndexDao.java --- @@ -282,4 +276,27 @@ public Document

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637474#comment-16637474 ] ASF GitHub Bot commented on METRON-1681: Github user merrimanr commented on the issue:

[GitHub] metron issue #1213: METRON-1681: Decouple the ParserBolt from the Parse exec...

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1213 Thanks for pointing me to that @mmiklavc, I had not looked at it yet. I am in agreement that we should do our best to match other classes and patterns we already have. I will continue studying

[GitHub] metron pull request #1213: METRON-1681: Decouple the ParserBolt from the Par...

Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1213#discussion_r222440594 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/ParserRunner.java --- @@ -0,0 +1,234 @@ +/** + * Licensed to

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637431#comment-16637431 ] ASF GitHub Bot commented on METRON-1681: Github user merrimanr commented on a diff in the pull

[jira] [Commented] (METRON-1804) Update version to 0.6.1

[ https://issues.apache.org/jira/browse/METRON-1804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637415#comment-16637415 ] ASF GitHub Bot commented on METRON-1804: Github user asfgit closed the pull request at:

[GitHub] metron pull request #1220: METRON-1804: Update version to 0.6.1

Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1220 ---

[jira] [Commented] (METRON-1761) Allow a grok statement to be applied to each line in a file.

[ https://issues.apache.org/jira/browse/METRON-1761?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637393#comment-16637393 ] ASF GitHub Bot commented on METRON-1761: Github user ottobackwards commented on the issue:

[GitHub] metron issue #1184: METRON-1761, allow application of grok statement multipl...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/1184 @mmiklavc but we don't have messages to split, we have bytes. If we where going to leave the 'parser's as single object -> single result | single exceception', ie not change the interface

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637376#comment-16637376 ] ASF GitHub Bot commented on METRON-1681: Github user nickwallen commented on the issue:

[GitHub] metron issue #1213: METRON-1681: Decouple the ParserBolt from the Parse exec...

Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1213 > @mmiklavc Have you looked over this by chance [2]? The reason I bring this up is because it looks like it already addresses a number of the points/questions brought up in this PR's discussion.

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637363#comment-16637363 ] ASF GitHub Bot commented on METRON-1681: Github user nickwallen commented on a diff in the pull

[GitHub] metron pull request #1213: METRON-1681: Decouple the ParserBolt from the Par...

Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1213#discussion_r222423266 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/ParserRunner.java --- @@ -0,0 +1,234 @@ +/** + * Licensed to

[jira] [Commented] (METRON-1771) Update REST endpoints to support eventually consistent UI updates

[ https://issues.apache.org/jira/browse/METRON-1771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637356#comment-16637356 ] ASF GitHub Bot commented on METRON-1771: Github user nickwallen commented on a diff in the pull

[GitHub] metron pull request #1190: METRON-1771: Update REST endpoints to support eve...

Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1190#discussion_r222421545 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/dao/ElasticsearchUpdateDaoTest.java --- @@ -0,0 +1,48 @@

[jira] [Commented] (METRON-1771) Update REST endpoints to support eventually consistent UI updates

[ https://issues.apache.org/jira/browse/METRON-1771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637352#comment-16637352 ] ASF GitHub Bot commented on METRON-1771: Github user nickwallen commented on a diff in the pull

[jira] [Commented] (METRON-1695) Expose pcap properties through Ambari

[ https://issues.apache.org/jira/browse/METRON-1695?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637354#comment-16637354 ] ASF GitHub Bot commented on METRON-1695: Github user anandsubbu commented on the issue:

[GitHub] metron issue #1207: METRON-1695: Expose pcap properties through Ambari

Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/1207 @nickwallen suggested a great idea of making use of parser aggregation and thus starting the default parsers on full-dev as an aggregated parser topology. The latest commit makes use of PR #1215

[GitHub] metron pull request #1190: METRON-1771: Update REST endpoints to support eve...

Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1190#discussion_r222420697 --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/MultiIndexDao.java --- @@ -282,4 +276,27 @@ public Document

[jira] [Commented] (METRON-1695) Expose pcap properties through Ambari

[ https://issues.apache.org/jira/browse/METRON-1695?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637343#comment-16637343 ] ASF GitHub Bot commented on METRON-1695: Github user anandsubbu commented on the issue:

[GitHub] metron issue #1207: METRON-1695: Expose pcap properties through Ambari

Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/1207 @MohanDV , as a part of my commit, I have moved all the UI related elements from REST to the PCAP config tab. In my understanding, the following parameters are not directly related to

[jira] [Commented] (METRON-1761) Allow a grok statement to be applied to each line in a file.

[ https://issues.apache.org/jira/browse/METRON-1761?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637336#comment-16637336 ] ASF GitHub Bot commented on METRON-1761: Github user mmiklavc commented on the issue:

[GitHub] metron issue #1184: METRON-1761, allow application of grok statement multipl...

Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/1184 @ottobackwards I see what you're saying. It looks like that could definitely work. Thinking out loud here, but might that conflate the semantics of our validation a bit? Validate currently does

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637313#comment-16637313 ] ASF GitHub Bot commented on METRON-1681: Github user mmiklavc commented on a diff in the pull

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637312#comment-16637312 ] ASF GitHub Bot commented on METRON-1681: Github user mmiklavc commented on a diff in the pull

[GitHub] metron pull request #1213: METRON-1681: Decouple the ParserBolt from the Par...

Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1213#discussion_r222376135 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/ParserRunner.java --- @@ -0,0 +1,234 @@ +/** + * Licensed to the

[GitHub] metron pull request #1213: METRON-1681: Decouple the ParserBolt from the Par...

Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/1213#discussion_r222397888 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/ParserRunner.java --- @@ -0,0 +1,234 @@ +/** + * Licensed to the

[jira] [Commented] (METRON-1804) Update version to 0.6.1

[ https://issues.apache.org/jira/browse/METRON-1804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637306#comment-16637306 ] ASF GitHub Bot commented on METRON-1804: Github user nickwallen commented on the issue:

[GitHub] metron issue #1220: METRON-1804: Update version to 0.6.1

Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1220 Looks good. Thanks! +1 ---

[jira] [Commented] (METRON-1804) Update version to 0.6.1

[ https://issues.apache.org/jira/browse/METRON-1804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637120#comment-16637120 ] ASF GitHub Bot commented on METRON-1804: GitHub user justinleet opened a pull request:

[GitHub] metron pull request #1220: METRON-1804: Update version to 0.6.1

GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1220 METRON-1804: Update version to 0.6.1 ## Contributor Comments Updating version to 0.6.1 post release. Changed per

[jira] [Created] (METRON-1804) Update version to 0.6.1

Justin Leet created METRON-1804: --- Summary: Update version to 0.6.1 Key: METRON-1804 URL: https://issues.apache.org/jira/browse/METRON-1804 Project: Metron Issue Type: Task

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637047#comment-16637047 ] ASF GitHub Bot commented on METRON-1681: Github user merrimanr commented on the issue:

[GitHub] metron issue #1213: METRON-1681: Decouple the ParserBolt from the Parse exec...

Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/1213 That sounds correct to me. Specifically decoupling the Kafka writing step since our writing implementation requires a tuple. ---

[jira] [Commented] (METRON-1681) Decouple the ParserBolt from the Parse execution logic

[ https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637041#comment-16637041 ] ASF GitHub Bot commented on METRON-1681: Github user mmiklavc commented on the issue:

[jira] [Commented] (METRON-1761) Allow a grok statement to be applied to each line in a file.

[ https://issues.apache.org/jira/browse/METRON-1761?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637040#comment-16637040 ] ASF GitHub Bot commented on METRON-1761: Github user ottobackwards commented on the issue:

[GitHub] metron issue #1184: METRON-1761, allow application of grok statement multipl...

Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/1184 @mmiklavc I looked through the validation stuff more, I think that validation is the way to go here. The grok parser will add invalid message for each exception, parser failure, and then in

[jira] [Commented] (METRON-1749) Update Angular to latest release in Management UI

[ https://issues.apache.org/jira/browse/METRON-1749?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16637036#comment-16637036 ] ASF GitHub Bot commented on METRON-1749: Github user sardell commented on the issue:

[GitHub] metron issue #1217: METRON-1749: Update Angular to latest release in Managem...

Github user sardell commented on the issue: https://github.com/apache/metron/pull/1217 @mmiklavc I was unable to find a discuss thread. I understand this is standard for a change like this and I would be more than happy to start one if the community desires, otherwise I'm hoping we

[jira] [Commented] (METRON-1796) [UI] Migrate off moment.js

[ https://issues.apache.org/jira/browse/METRON-1796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636991#comment-16636991 ] ASF GitHub Bot commented on METRON-1796: Github user ruffle1986 commented on a diff in the pull

[jira] [Commented] (METRON-1796) [UI] Migrate off moment.js

[ https://issues.apache.org/jira/browse/METRON-1796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636992#comment-16636992 ] ASF GitHub Bot commented on METRON-1796: Github user ruffle1986 commented on a diff in the pull

[GitHub] metron pull request #1219: METRON-1796: [UI] Migrate off moment.js

Github user ruffle1986 commented on a diff in the pull request: https://github.com/apache/metron/pull/1219#discussion_r222315533 --- Diff: metron-interface/metron-alerts/package.json --- @@ -46,9 +46,7 @@ "@types/ace": "0.0.32", "@types/es6-promise": "0.0.33",

[GitHub] metron pull request #1219: METRON-1796: [UI] Migrate off moment.js

Github user ruffle1986 commented on a diff in the pull request: https://github.com/apache/metron/pull/1219#discussion_r222317156 --- Diff: metron-interface/metron-alerts/src/app/utils/utils.spec.ts --- @@ -0,0 +1,215 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] metron pull request #1219: METRON-1796: [UI] Migrate off moment.js

Github user ruffle1986 commented on a diff in the pull request: https://github.com/apache/metron/pull/1219#discussion_r222308215 --- Diff: metron-interface/metron-alerts/package.json --- @@ -22,17 +22,17 @@ "@angular/platform-browser": "^6.1.6",

[GitHub] metron issue #1217: METRON-1749: Update Angular to latest release in Managem...

Github user sardell commented on the issue: https://github.com/apache/metron/pull/1217 @mmiklavc Thanks for starting to take a look at this PR! You're correct that all these changes are related to the upgrade. This means removing deprecated items, updating syntax, utilizing

[jira] [Commented] (METRON-1749) Update Angular to latest release in Management UI

[ https://issues.apache.org/jira/browse/METRON-1749?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636951#comment-16636951 ] ASF GitHub Bot commented on METRON-1749: Github user sardell commented on the issue:

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636948#comment-16636948 ] ASF GitHub Bot commented on METRON-1798: Github user asfgit closed the pull request at:

[GitHub] metron pull request #1215: METRON-1798: Add mpack support for parser aggrega...

Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1215 ---

[jira] [Commented] (METRON-1796) [UI] Migrate off moment.js

[ https://issues.apache.org/jira/browse/METRON-1796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636946#comment-16636946 ] ASF GitHub Bot commented on METRON-1796: GitHub user ruffle1986 opened a pull request:

[GitHub] metron pull request #1219: METRON-1796: [UI] Migrate off moment.js

GitHub user ruffle1986 opened a pull request: https://github.com/apache/metron/pull/1219 METRON-1796: [UI] Migrate off moment.js ## Contributor Comments [DISCUSS] thread on the dev mailing list:

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636945#comment-16636945 ] ASF GitHub Bot commented on METRON-1798: Github user anandsubbu commented on the issue:

[GitHub] metron issue #1215: METRON-1798: Add mpack support for parser aggregation

Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/1215 Thanks again for the review, @nickwallen ! ---

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636819#comment-16636819 ] ASF GitHub Bot commented on METRON-1798: Github user nickwallen commented on the issue:

[GitHub] metron issue #1215: METRON-1798: Add mpack support for parser aggregation

Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1215 +1 This is really good fix @anandsubbu . Thanks for the easily grokable docs and clean code. ---

[jira] [Created] (METRON-1803) Integrate Cypress tests with Travis

Tibor Meller created METRON-1803: Summary: Integrate Cypress tests with Travis Key: METRON-1803 URL: https://issues.apache.org/jira/browse/METRON-1803 Project: Metron Issue Type: Improvement

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636695#comment-16636695 ] ASF GitHub Bot commented on METRON-1798: GitHub user anandsubbu reopened a pull request:

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636692#comment-16636692 ] ASF GitHub Bot commented on METRON-1798: Github user anandsubbu closed the pull request at:

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636694#comment-16636694 ] ASF GitHub Bot commented on METRON-1798: Github user anandsubbu commented on the issue:

[GitHub] metron issue #1215: METRON-1798: Add mpack support for parser aggregation

Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/1215 Re-run travis ---

[GitHub] metron pull request #1215: METRON-1798: Add mpack support for parser aggrega...

GitHub user anandsubbu reopened a pull request: https://github.com/apache/metron/pull/1215 METRON-1798: Add mpack support for parser aggregation ## Contributor Comments This pull request allows users to submit an aggregated parser topology. ## Testing Steps 1. Spin

[GitHub] metron pull request #1215: METRON-1798: Add mpack support for parser aggrega...

Github user anandsubbu closed the pull request at: https://github.com/apache/metron/pull/1215 ---

[jira] [Commented] (METRON-1798) Add mpack support for parser aggregation

[ https://issues.apache.org/jira/browse/METRON-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16636668#comment-16636668 ] ASF GitHub Bot commented on METRON-1798: Github user anandsubbu commented on the issue:

[GitHub] metron issue #1215: METRON-1798: Add mpack support for parser aggregation

Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/1215 Hi @nickwallen , thank you for the review. > @nickwallen One thing I notice right off is that we did not add any documentation for the parsers fields in Ambari. Would it make sense to

[jira] [Updated] (METRON-1802) When switching parser topologies, some of the original parser topologies can fail to be shut down properly

[ https://issues.apache.org/jira/browse/METRON-1802?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anand Subramanian updated METRON-1802: -- Description: This defect was reported by [~nickwallen] when testing PR[

[jira] [Created] (METRON-1802) When switching parser topologies, some of the original parser topologies can fail to be shut down properly

Anand Subramanian created METRON-1802: - Summary: When switching parser topologies, some of the original parser topologies can fail to be shut down properly Key: METRON-1802 URL: