Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2698
Thanks for addressing the comments. Looks good to me. I think it makes
sense to add shaded-deps as a module to the root pom though. It will likely
address the release issue, and means we don't have to manually keep its version
number in sync with the rest of Storm. Is there a reason to not have it listed
in the root?---
