-
Notifications
You must be signed in to change notification settings - Fork 8k
Ensure a sufficient virtual threads pool #37197
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.
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
Ensure a sufficient virtual threads pool #37197
Conversation
afff2db to
c457ef6
Compare
| This reduces the memory usage and removes the need to configure thread pool sizes. | ||
| Therefore, the use of OpenJDK 21 is recommended. | ||
|
|
||
| === JGroups communications |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems duplicated from line 15.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for spotting this one, fixed. Please re-review.
c457ef6 to
c692dcb
Compare
Closes keycloak#37162 Signed-off-by: Alexander Schwartz <[email protected]>
c692dcb to
6bf562d
Compare
mabartos
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good from the Quarkus code side.
mhajas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving based on reviews from others
Closes #37162