Github user Ethanlm commented on a diff in the pull request: https://github.com/apache/storm/pull/2754#discussion_r208377360 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/timer/ReportWorkerHeartbeats.java --- @@ -51,6 +51,7 @@ public void run() { private SupervisorWorkerHeartbeats getAndResetWorkerHeartbeats() { Map<String, LSWorkerHeartbeat> localHeartbeats; try { + //TODO: This call no longer throws exceptions, do we still want to wrap it in try catch block? --- End diff -- remove comment
---