Github user pczb commented on a diff in the pull request:
https://github.com/apache/storm/pull/2669#discussion_r188666612
--- Diff: storm-client/src/jvm/org/apache/storm/messaging/IContext.java ---
@@ -38,6 +38,7 @@
/**
* This method establishes a server side connection
+ * implementation should return a new connection every call
--- End diff --
remove bind specification, since the new and old netty context never cache
server connections, server connections is add to array or map just for cleanup
---
