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.

Sign up for GitHub

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

Jump to bottom

EventLoopProcessor#shutdownNow should also shutdown the request task #1074

Closed
simonbasle opened this issue Feb 14, 2018 · 0 comments
Closed

EventLoopProcessor#shutdownNow should also shutdown the request task #1074

simonbasle opened this issue Feb 14, 2018 · 0 comments
Assignees
@simonbasle
Labels
type/bug A general bug
Milestone
3.1.4.RELEASE

Comments

@simonbasle
Copy link
Member

Expected behavior

After a shutdownNow(), both the executor and the requestTaskExecutor should be terminated.

Actual behavior

Only the executor is terminated.
@simonbasle simonbasle added the type/bug A general bug label Feb 14, 2018
@simonbasle simonbasle added this to the 3.1.4.RELEASE milestone Feb 14, 2018
@simonbasle simonbasle self-assigned this Feb 14, 2018
simonbasle added a commit that referenced this issue Feb 14, 2018
@simonbasle
fix #1074 Shutdown request task executor in event loop's forceShutdown
e4b6157 
The EventLoopProcessors would fail to shut down their request task
executor when `forceShutdown` was called.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simonbasle simonbasle

Labels
type/bug A general bug
Projects
None yet
Milestone
3.1.4.RELEASE
Development

No branches or pull requests
1 participant
@simonbasle