Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2669#discussion_r188377925
--- 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 --
Bind doesn't always return a new connection, the local Context will return
a cached one if possible. Consider changing to something like "This method
returns a server side connection. If one does not exist for the given ID and
port, a new one will be established."
---
