Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STORM-2336 Close Localizer and AsyncLocalizer when supervisor is shutting down #1911

Merged
merged 1 commit into from
Feb 2, 2017

Conversation

HeartSaVioR
Copy link
Contributor

  • this patch gets rid of alive non-daemon threads which prevent JVM process shutdown

This patch can be ported back to 1.x and 1.0.x branch via cherry-picking.

…ting down

* this patch gets rid of alive non-daemon threads which prevent JVM process shutdown
@HeartSaVioR HeartSaVioR changed the title STORM-2236 Close Localizer and AsyncLocalizer when supervisor is shutting down STORM-2336 Close Localizer and AsyncLocalizer when supervisor is shutting down Feb 1, 2017
@revans2
Copy link
Contributor

revans2 commented Feb 1, 2017

+1

2 similar comments
@harshach
Copy link
Contributor

harshach commented Feb 1, 2017

+1

@hmcl
Copy link
Contributor

hmcl commented Feb 2, 2017

+1

@asfgit asfgit merged commit 44cfe3d into apache:master Feb 2, 2017
@tibkiss
Copy link
Contributor

tibkiss commented Feb 14, 2017

For future reference:
This issue has been committed with a typo in the issue id:
STORM-2236 was used as opposed to the correct STORM-2336

64a9dd0 2017-02-01 Jungtaek Lim STORM-2236 Close Localizer and AsyncLocalizer when supervisor is shutting down

@HeartSaVioR HeartSaVioR deleted the STORM-2336 branch June 29, 2017 14:39
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.

6 participants