Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: prevent blocking event loop thread #2936

Closed
wants to merge 1 commit into from
Closed

fix: prevent blocking event loop thread #2936

wants to merge 1 commit into from

Conversation

leisurelyrcxf
Copy link

@leisurelyrcxf leisurelyrcxf commented Aug 1, 2024 •
edited
Loading

refers to issue: #2937

Make sure that:

  • [Y] You have read the contribution guidelines.
  • [Y] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • [Y] You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • [Y] You submit test cases (unit or integration tests) that back your changes.

@leisurelyrcxf leisurelyrcxf force-pushed the fix/avoid-long-block-during-disconnection branch 8 times, most recently from b218212 to f5839be Compare August 1, 2024 10:53
@leisurelyrcxf leisurelyrcxf changed the title perf: use LinkedHashSetQueue instead of ArrayDeque for CommandHandler… fix: prevent blocking event loop thread Aug 1, 2024
@leisurelyrcxf leisurelyrcxf force-pushed the fix/avoid-long-block-during-disconnection branch from f5839be to d300391 Compare August 1, 2024 11:29
@okg-cxf
fix: prevent long stall during reconnection/disconnection by providin…
2acda0d 
…g LinkedHashSetQueue instead of ArrayDeque for CommandHandler#stack
@leisurelyrcxf leisurelyrcxf force-pushed the fix/avoid-long-block-during-disconnection branch from d300391 to 2acda0d Compare August 1, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leisurelyrcxf @okg-cxf