You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by revans2 <gi...@git.apache.org> on 2018/10/08 18:28:22 UTC
[GitHub] storm pull request #2867: STORM-3249: Make sure times shut down and so does ...
GitHub user revans2 opened a pull request:
https://github.com/apache/storm/pull/2867
STORM-3249: Make sure times shut down and so does shut down thread
Turns out there are two things blocking nimbus from shutting down.
1) some timer (that turns out to be from the local fs blobstore.
2) The thread that was waiting for the timeout before calling halt was also stopping us from shutting down :)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/revans2/incubator-storm STORM-3249
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2867.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2867
----
commit bbfd7cd4050ae80d406daaed9cec882b00be1f98
Author: Robert (Bobby) Evans <ev...@...>
Date: 2018-10-08T18:25:17Z
STORM-3249: Make sure times shut down and so does shut down thread
----
---
[GitHub] storm pull request #2867: STORM-3249: Make sure times shut down and so does ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2867
---