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.
---
