[GitHub] metron issue #732: METRON-632: Added validation of "shew.enrichmentType" and...

2017-09-27 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/732 Thank you! ---

[GitHub] metron issue #732: METRON-632: Added validation of "shew.enrichmentType" and...

2017-09-26 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/732 @cestella Could I possibly get this approved? Cheers ---

[GitHub] metron pull request #732: METRON-632: Added validation of "shew.enrichmentTy...

2017-09-10 Thread zezutom
Github user zezutom commented on a diff in the pull request: https://github.com/apache/metron/pull/732#discussion_r137960020 --- Diff: metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/writer/SimpleHbaseEnrichmentWriter.java --- @@ -134,6 +136,39

[GitHub] metron pull request #732: METRON-632: Added validation of "shew.enrichmentTy...

2017-09-03 Thread zezutom
GitHub user zezutom opened a pull request: https://github.com/apache/metron/pull/732 METRON-632: Added validation of "shew.enrichmentType" and "shew.keyCo… …lumns" ## Contributor Comments [Please place any comments here. A description of the

[GitHub] metron issue #572: METRON-633: Create better logging for HbaseEnrichmentWrit...

2017-07-05 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/572 @justinleet I can't merge this PR, do you mind merging to master yourself? Cheers. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] metron issue #572: METRON-633: Create better logging for HbaseEnrichmentWrit...

2017-07-05 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/572 @justinleet Will do, thanks for sorting out Travis! --- 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

[GitHub] metron issue #572: METRON-633: Create better logging for HbaseEnrichmentWrit...

2017-06-18 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/572 Thank you @cestella and sorry for the delay, now I am back to it. --- 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

[GitHub] metron pull request #572: METRON-633: Create better logging for HbaseEnrichm...

2017-06-01 Thread zezutom
Github user zezutom commented on a diff in the pull request: https://github.com/apache/metron/pull/572#discussion_r119730181 --- Diff: metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/writer/SimpleHbaseEnrichmentWriter.java --- @@ -169,16 +188,19

[GitHub] metron issue #572: METRON-633: Create better logging for HbaseEnrichmentWrit...

2017-05-18 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/572 Hey folks, this one has been open for a while. Could I please get some +1's? Thank you! --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] metron issue #558: METRON-811: Enforce Maven Version in Top Level POM

2017-05-18 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/metron/pull/558 Thanks @ottobackwards. Anyone else, can I please get one more thumb up to get this PR over the line? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-metron pull request #572: METRON-633: Create better logging for Hb...

2017-05-08 Thread zezutom
GitHub user zezutom opened a pull request: https://github.com/apache/incubator-metron/pull/572 METRON-633: Create better logging for HbaseEnrichmentWriter ## Contributor Comments I have added logging to critical paths, so that issues can be tracked down more easily

[GitHub] incubator-metron pull request #571: METRON-633: Create better logging for Hb...

2017-05-08 Thread zezutom
Github user zezutom closed the pull request at: https://github.com/apache/incubator-metron/pull/571 --- 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

[GitHub] incubator-metron issue #571: METRON-633: Create better logging for HbaseEnri...

2017-05-08 Thread zezutom
Github user zezutom commented on the issue: https://github.com/apache/incubator-metron/pull/571 The build is timing out for some reason, I will resubmit this PR. --- 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

[GitHub] incubator-metron pull request #571: METRON-633: Create better logging for Hb...

2017-05-08 Thread zezutom
GitHub user zezutom reopened a pull request: https://github.com/apache/incubator-metron/pull/571 METRON-633: Create better logging for HbaseEnrichmentWriter ## Contributor Comments I have added logging to critical paths, so that issues can be tracked down more easily

[GitHub] incubator-metron pull request #571: METRON-633: Create better logging for Hb...

2017-05-08 Thread zezutom
Github user zezutom closed the pull request at: https://github.com/apache/incubator-metron/pull/571 --- 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

[GitHub] incubator-metron pull request #571: METRON-633: Create better logging for Hb...

2017-05-08 Thread zezutom
GitHub user zezutom opened a pull request: https://github.com/apache/incubator-metron/pull/571 METRON-633: Create better logging for HbaseEnrichmentWriter ## Contributor Comments I have added logging to critical paths, so that issues can be tracked down more easily

[GitHub] incubator-metron pull request #558: METRON-811: Enforce Maven Version in Top...

2017-04-29 Thread zezutom
GitHub user zezutom opened a pull request: https://github.com/apache/incubator-metron/pull/558 METRON-811: Enforce Maven Version in Top Level POM ## Contributor Comments Enforcing Maven version >= 3.3.1 and Java 8 ## Pull Request Checklist Thank you

[GitHub] incubator-metron pull request #557: METRON-811: Enforce Maven Version in Top...

2017-04-28 Thread zezutom
Github user zezutom closed the pull request at: https://github.com/apache/incubator-metron/pull/557 --- 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

[GitHub] incubator-metron pull request #557: METRON-811: Enforce Maven Version in Top...

2017-04-28 Thread zezutom
GitHub user zezutom opened a pull request: https://github.com/apache/incubator-metron/pull/557 METRON-811: Enforce Maven Version in Top Level POM ## Contributor Comments Enforcing Maven version >= 3.3.1 and Java 8 ## Pull Request Checklist Thank you