[GitHub] metron issue #599: METRON-975: Normalize logging and switch to common idiom ...

2017-07-17 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/599 @merrimanr Conflicts resolved --- 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

[GitHub] metron pull request #620: Metron-988: UI for viewing alerts generated by Met...

2017-07-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/620#discussion_r127832531 --- Diff: metron-interface/metron-alerts/src/app/service/data-source.ts --- @@ -0,0 +1,62 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] metron issue #647: METRON-1031: Management UI Cannot Start Topologies in Ker...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/647 +1 validated with instructions in the PR description. Nice work! --- 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

[GitHub] metron issue #599: METRON-975: Normalize logging and switch to common idiom ...

2017-07-17 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/599 I would love to get this in --- 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

[GitHub] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/645 Their status will "Active" but you'll see Kerberos errors in the Storm UI. --- 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] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/645 Without it, the enrichment and indexing topologies don't work on a Kerberized environment because the Storm worker childopts property needs the

[GitHub] metron issue #645: METRON-1029: Topology worker opts are not being set when ...

2017-07-17 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/645 What happens without this PR? What behavior are we correcting? --- 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

[GitHub] metron issue #647: METRON-1031: Management UI Cannot Start Topologies in Ker...

2017-07-17 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/647 @merrimanr Ran it through a full test as described in the PR description and its looking good. Please take a look when you can. --- If your project is set up for it, you can reply to this

[GitHub] metron issue #359: METRON-569: Change acking to prevent duplicate tuples in ...

2017-07-17 Thread DomenicPuzio
Github user DomenicPuzio commented on the issue: https://github.com/apache/metron/pull/359 Yep, we can wrap it up! Thanks, all! --- 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

[GitHub] metron pull request #359: METRON-569: Change acking to prevent duplicate tup...

2017-07-17 Thread DomenicPuzio
Github user DomenicPuzio closed the pull request at: https://github.com/apache/metron/pull/359 --- 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 feature

[GitHub] metron issue #359: METRON-569: Change acking to prevent duplicate tuples in ...

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/359 @DomenicPuzio, this has been resolved by https://github.com/apache/metron/pull/603. Can we close this PR? --- If your project is set up for it, you can reply to this email and have your reply

Re: Elasticsearch 5.x upgrade

2017-07-17 Thread Otto Fowler
The plan around the mpack with regards to elastic search and solr support and options is extremely important and as of yet undecided. On July 17, 2017 at 03:00:42, Simon Elliston Ball ( si...@simonellistonball.com) wrote: > I assume you're taking about METRON-939. The upgrade is not a huge

Re: Elasticsearch 5.x upgrade

2017-07-17 Thread Simon Elliston Ball
I assume you're taking about METRON-939. The upgrade is not a huge change, and unlikely to touch any of the bulk writing pieces. The main thing outstanding on the PR is log4j dependency conflicts from the new ES client. There's no work in that PR to do anything around mpack based install either