Skip to content
This repository has been archived by the owner on Mar 15, 2021. It is now read-only.

Lower thread pool timeout #105

Open
aikar opened this issue Apr 10, 2020 · 1 comment
Open

Lower thread pool timeout #105

aikar opened this issue Apr 10, 2020 · 1 comment

Comments

@aikar
Copy link

aikar commented Apr 10, 2020

ExecutorThreadPool pool = new ExecutorThreadPool(2, maxconnections, 60, TimeUnit.SECONDS, queue);

I noticed in debugger that I had a ton of "etp" threads. Traced it down to jetty from dynmap.

The code times them out in 60 seconds, but I'm afraid that each thread is eventually popping 1 task off within that 60 seconds to keep each thread alive. I do not see threads closing....

suggest dropping it to like 1-5 seconds so that they can close and only open on burst traffic.

@aikar
Copy link
Author

aikar commented Jun 12, 2020

@mikeprimm saw a thread dump that reminded me of this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant