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?
---