Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2651#discussion_r189420057 --- Diff: storm-client/src/jvm/org/apache/storm/drpc/ReturnResults.java --- @@ -103,21 +85,32 @@ public void execute(Tuple input) { LOG.error("Failed to return results to DRPC server", tex); _collector.fail(input); } - reconnectClient((DRPCInvocationsClient) client); + client = getDRPCClient(host, port); --- End diff -- @kishorvpatil I meant we don't invalidate broken `client` from `_clients` so it will always pick same `client`, instead of rebuilding new `client`.
---