Skip to content

fix: make WebSocket session sends asynchronous (#6568) - #6980

Open
yykaue wants to merge 1 commit into
apache:masterfrom
yykaue:fix-6568-async-websocket-send
Open

fix: make WebSocket session sends asynchronous (#6568)#6980
yykaue wants to merge 1 commit into
apache:masterfrom
yykaue:fix-6568-async-websocket-send

Conversation

@yykaue

@yykaue yykaue commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the global synchronization from WebSocket message sending.
  • Switch from blocking BasicRemote.sendText to asynchronous AsyncRemote.sendText.
  • Add per-session send queues to preserve message ordering without blocking other gateway sessions.
  • Clean up send queues when sessions are closed or encounter errors.
  • Add tests for cross-session non-blocking behavior and per-session message ordering.

Fixes #6568.

@Aias00 Could you please help review this PR? Thanks!

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.

[BUG] WebsocketCollector.sendMessageBySession: global synchronized + blocking synchronous send

1 participant