Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ahus1
Copy link
Contributor

@ahus1 ahus1 commented Feb 10, 2025

Closes #37162

@ahus1 ahus1 self-assigned this Feb 10, 2025
@ahus1 ahus1 force-pushed the is-37162-ensure-larger-vt-thread-pool branch 2 times, most recently from afff2db to c457ef6 Compare February 10, 2025 10:49
@ahus1
Copy link
Contributor Author

ahus1 commented Feb 10, 2025

Hi @pruivo - see 344b653 for my first attempt. I now updated it to avoid virtual threads where only one core is available. Please let me know your thoughts.

Thanks!

@ahus1 ahus1 marked this pull request as ready for review February 10, 2025 13:25
@ahus1 ahus1 requested review from a team as code owners February 10, 2025 13:25
This reduces the memory usage and removes the need to configure thread pool sizes.
Therefore, the use of OpenJDK 21 is recommended.

=== JGroups communications
Copy link
Member

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.

Copy link
Contributor Author

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.

@ahus1 ahus1 force-pushed the is-37162-ensure-larger-vt-thread-pool branch from c457ef6 to c692dcb Compare February 10, 2025 13:54
@ahus1 ahus1 requested a review from pruivo February 10, 2025 13:54
@ahus1 ahus1 force-pushed the is-37162-ensure-larger-vt-thread-pool branch from c692dcb to 6bf562d Compare February 10, 2025 13:55
Copy link
Contributor

@mabartos mabartos left a 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.

Copy link
Contributor

@mhajas mhajas left a 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

@mhajas mhajas merged commit 822eb44 into keycloak:main Feb 10, 2025
77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pods become unresponsive after upgrade to 26.1.0

4 participants