[GitHub] storm pull request #2618: [STORM-2905] Fix KeyNotFoundException when kill a ...

2018-04-03 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2618#discussion_r178852812 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/Slot.java --- @@ -497,7 +497,6 @@ static DynamicState cleanupCurrentContainer(Dynami

[GitHub] storm pull request #2618: [STORM-2905] Fix KeyNotFoundException when kill a ...

2018-04-02 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2618#discussion_r178693115 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocalizedResourceRetentionSet.java --- @@ -94,17 +93,17 @@ public void cleanup(ClientBlobS

[GitHub] storm pull request #2618: [STORM-2905] Fix KeyNotFoundException when kill a ...

2018-04-02 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2618#discussion_r178692521 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/Slot.java --- @@ -497,7 +497,6 @@ static DynamicState cleanupCurrentContainer(Dyna

[GitHub] storm pull request #2618: [STORM-2905] Fix KeyNotFoundException when kill a ...

2018-04-02 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2618#discussion_r178568732 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocalizedResourceRetentionSet.java --- @@ -94,17 +93,17 @@ public void cleanup(ClientBlobSto

[GitHub] storm pull request #2618: [STORM-2905] Fix KeyNotFoundException when kill a ...

2018-04-02 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2618#discussion_r178567096 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/supervisor/Slot.java --- @@ -497,7 +497,6 @@ static DynamicState cleanupCurrentContainer(Dynami

[GitHub] storm pull request #2618: [STORM-2905] Fix KeyNotFoundException when kill a ...

2018-03-31 Thread danny0405
GitHub user danny0405 opened a pull request: https://github.com/apache/storm/pull/2618 [STORM-2905] Fix KeyNotFoundException when kill a storm and add isRem… When we kill a topology, at the moment of topology blob-files be removed, Supervisor executor still request blob-files and