Github user justinleet commented on the issue: https://github.com/apache/metron/pull/1217 While admitting some ignorance of proper formatting and unfortunately even best practices, I'm pretty good with the changes, particularly with @tiborm's +1. I know there's the checkbox, but I did want to double check that all the dependencies hit the license reqs. In particular, that nothing that had to be upgraded changed licenses to something incompatible. Given the large version leap, I think it bears calling out specifically. @sardell Did you run through that exercise?
---