Github user zd-project commented on a diff in the pull request: https://github.com/apache/storm/pull/2710#discussion_r208286166 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/Container.java --- @@ -346,6 +358,10 @@ public void cleanUp() throws IOException { _usedMemory.remove(_port); _reservedMemory.remove(_port); cleanUpForRestart(); + } catch (IOException e) { + //This may or may not be reported depending on when process exits --- End diff -- My bad. I thought Container runs on it's own process when I started working on this. This should not be a problem if it's just a daemon thread.
---