Skip to content

STORM-3249: Make sure times shut down and so does shut down thread#2867

Merged
asfgit merged 1 commit intoapache:masterfrom
revans2:STORM-3249
Oct 9, 2018
Merged

STORM-3249: Make sure times shut down and so does shut down thread#2867
asfgit merged 1 commit intoapache:masterfrom
revans2:STORM-3249

Conversation

@revans2
Copy link
Contributor

@revans2 revans2 commented Oct 8, 2018

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 :)

Copy link
Contributor

@kishorvpatil kishorvpatil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@asfgit asfgit merged commit bbfd7cd into apache:master Oct 9, 2018
asfgit pushed a commit that referenced this pull request Oct 9, 2018
… into STORM-3249

STORM-3249: Make sure times shut down and so does shut down thread

This closes #2867
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants