Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2669#discussion_r188055495
--- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/Client.java
---
@@ -451,7 +451,6 @@ public int getPort() {
public void close() {
if (!closing) {
LOG.info("closing Netty Client {}", dstAddressPrefixedName);
- context.removeClient(dstHost, dstAddress.getPort());
--- End diff --
This line is the only reason to pass context into the Client. Please
remove all references to context if we are going to remove this.
---
