Skip to content

Drop virtual threads in ScheduledExecutorService#928

Merged
jansupol merged 1 commit intoeclipse-ee4j:2.0.xfrom
jansupol:scheduled.services
Jun 28, 2025
Merged

Drop virtual threads in ScheduledExecutorService#928
jansupol merged 1 commit intoeclipse-ee4j:2.0.xfrom
jansupol:scheduled.services

Conversation

@jansupol
Copy link
Copy Markdown
Contributor

@jansupol jansupol commented Jun 4, 2025

No description provided.

@jglick
Copy link
Copy Markdown

jglick commented Jun 24, 2025

#927 (comment) So was the ScheduledExecutorService actually used in practice? I am trying to understand whether the key problem motivating #669

If we use a ClientManager per connection the thread usage [for ~150 connections] is way [too] high (10 schedule-exec threads + 1-2 worker-threads = ~1800 threads)

remains resolved even with this PR. I am currently investigating a rewrite to the built-in WS client since Tyrus 2.2.0 consumes far too many threads.

@jansupol jansupol merged commit c3bf45f into eclipse-ee4j:2.0.x Jun 28, 2025
3 checks passed
@jansupol jansupol deleted the scheduled.services branch June 28, 2025 12:15
@jansupol
Copy link
Copy Markdown
Contributor Author

jansupol commented Jul 7, 2025

@jglick These ScheduledExecutorServices are used for PING-PONG heartbeat messages only.

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