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

2017-08-01 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/599 +1 by inspection --- 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

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

2017-07-31 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/metron/pull/599 +1 (non-binding) the sooner we get this in the better. It has performance benefits, and the longer we wait the more work and trouble it will create. No reason not to get it done, lots to ge

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

2017-07-31 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/599 Any comments on this? I've been merging/resolving conflicts with master as they come in. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

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

2017-07-18 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/599 I saw this same test failure yesterday on another PR. I opened a [METRON-1048](https://issues.apache.org/jira/browse/METRON-1048) for it. --- If your project is set up for it, you can reply to

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

2017-07-18 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/599 I'm damned confused about these test errors. This PR didn't seem to touch anything near the test and I haven't seen that test (which is a mock test) fail intermittently before. I'd love a debrief

[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 w

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

2017-07-17 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/599 @mmiklavc looks like there are some merge conflicts. If you don't mind resolving those I will get this reviewed. --- If your project is set up for it, you can reply to this email and have your re

[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 wis

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

2017-07-07 Thread simonellistonball
Github user simonellistonball commented on the issue: https://github.com/apache/metron/pull/599 Is it worth us getting this in sooner rather than later, before we get too many other bits of logging that will need to be backported? --- If your project is set up for it, you can reply

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

2017-05-31 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/599 I will give you real imaginary money to rebase this onto METRON-777 so I don't have to port ;) --- If your project is set up for it, you can reply to this email and have your reply appear on G

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

2017-05-31 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/599 Kick Travis - rest api test failure --- 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