- Notifications
You must be signed in to change notification settings - Fork 38.8k
Closed as not planned
Labels
in: webIssues in web modules (web, webmvc, webflux, websocket)Issues in web modules (web, webmvc, webflux, websocket)status: supersededAn issue that has been superseded by anotherAn issue that has been superseded by another
Description
#30996 fixed thread pinning issue. But only for SSEEmitter. Still there are several synchronized block in ResponseBodyEmitter.
Line 183 in e5b58ef
public synchronized void send(Object object, @Nullable MediaType mediaType) throws IOException { |
Line 211 in e5b58ef
public synchronized void send(Set<DataWithMediaType> items) throws IOException { |
To prevent thread pinning when using virtual thread, it have to replaced with ReentrantLock
.
Many of us cannot move to JDK 24 yet (which reduces synchronized pinning), so we need a framework-level mitigation.
Metadata
Metadata
Assignees
Labels
in: webIssues in web modules (web, webmvc, webflux, websocket)Issues in web modules (web, webmvc, webflux, websocket)status: supersededAn issue that has been superseded by anotherAn issue that has been superseded by another