-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Stopping ReservedThreadExecutor may hang #4575
Comments
@gregw ideas? |
@sbordet I think the problem is that a reserved thread can be stopped either by: it sees that isRunning is false; or it is passed a poison pill. The solution is that we should only ever stop a reserved thread with a poison pill, so that way it will always be polling the synchronous queue and we will never block forever handing over a task. |
This is likely to be a problem in jetty-9 also! |
See #4576 |
…ved-threads Fixes #4575 Stopping Reserved Thread
Jetty version
10.0.x
Java version
13
Description
Running the test suite, test
org.eclipse.jetty.embedded.ExampleServerTest
may result in theJVM hanging during stop:What above is trying to stop
HttpClient
, and itsQueuedThreadPool
, but theReservedThreadExecutor
seems to hang waiting for a thread, although there seems to be threads available in the pool (thoseclient-*
threads).The text was updated successfully, but these errors were encountered: