Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2704#discussion_r194141182
--- Diff: shaded-deps/pom.xml ---
@@ -251,6 +253,10 @@
<pattern>org.jboss.netty</pattern>
<shadedPattern>org.apache.storm.shade.org.jboss.netty</shadedPattern>
</relocation>
+ <relocation>
--- End diff --
Zookeeper depends on Netty 3. I think we still need to relocate it, even
though our code doesn't use it directly.---
