[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

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

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203115089 --- Diff: use-cases/parser_chaining/README.md --- @@ -233,3 +233,10 @@ cat ~/data.log | /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh --b

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203095632 --- Diff: use-cases/parser_chaining/README.md --- @@ -233,3 +233,10 @@ cat ~/data.log | /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203092801 --- Diff: use-cases/parser_chaining/README.md --- @@ -233,3 +233,10 @@ cat ~/data.log | /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh --b

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203083284 --- Diff: use-cases/parser_chaining/README.md --- @@ -233,3 +233,10 @@ cat ~/data.log | /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203064655 --- Diff: use-cases/parser_chaining/README.md --- @@ -233,3 +233,10 @@ cat ~/data.log | /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203052754 --- Diff: use-cases/parser_chaining/aggregated_parser_chaining_flow.xml --- @@ -0,0 +1 @@

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203052624 --- Diff: use-cases/parser_chaining/aggregated_parser_chaining_flow.svg --- @@ -0,0 +1,2 @@ +http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd;> ---

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-17 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r203050720 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202818595 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202817242 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202814185 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202813546 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202812681 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202808756 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202805609 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202805243 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202803869 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/bolt/ParserBolt.java --- @@ -182,40 +185,61 @@ public void prepare(Map

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202803106 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202802349 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/bolt/ParserBolt.java --- @@ -182,40 +185,61 @@ public void

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202801374 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202798006 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/bolt/ParserBolt.java --- @@ -182,40 +185,61 @@ public void

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202797418 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202785248 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/bolt/ParserBolt.java --- @@ -182,40 +185,61 @@ public void prepare(Map

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202755740 --- Diff: metron-platform/metron-parsers/README.md --- @@ -82,6 +82,12 @@ topology in kafka. Errors are collected with the context of the error

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-16 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r202758396 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/bolt/ParserBolt.java --- @@ -182,40 +185,61 @@ public void

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-10 Thread justinleet
Github user justinleet closed the pull request at: https://github.com/apache/metron/pull/1099 ---

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-10 Thread justinleet
GitHub user justinleet reopened a pull request: https://github.com/apache/metron/pull/1099 METRON-1657: Parser aggregation in storm ## Contributor Comments This PR allows for users to specify multiple parsers to be run in one aggregated Storm topology. Essentially, the

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-10 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r201361910 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/topology/ParserTopologyBuilder.java --- @@ -91,14 +101,14 @@ public

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-10 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r201356953 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/topology/ParserTopologyBuilder.java --- @@ -91,14 +101,14 @@ public

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-10 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1099#discussion_r201349285 --- Diff: metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/topology/ParserTopologyBuilder.java --- @@ -91,14 +101,14 @@ public

[GitHub] metron pull request #1099: METRON-1657: Parser aggregation in storm

2018-07-10 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1099 METRON-1657: Parser aggregation in storm ## Contributor Comments This PR allows for users to specify multiple parsers to be run in one aggregated Storm topology. Essentially, the