-
Notifications
You must be signed in to change notification settings - Fork 8k
Closed
Labels
kind/bugCategorizes a PR related to a bugCategorizes a PR related to a bugpriority/importantMust be worked on very soonMust be worked on very soonrelease/26.0.11release/26.1.3release/26.2.0team/sre
Description
Description
The latest JGroups patch version avoid thread pinning in several places, with JDBC_PING2 being probably the most prominent.
- JGRP-2861 Avoiding JDBC_PING2 synchronized as it would pin the threads when using virtual threads belaban/JGroups#877
- JGRP-2864 Avoiding synchronized in BaseServer as it would pin the threads when using virtual threads belaban/JGroups#878
- JGRP-2863 Avoiding synchronized in GMS.installView as it would pin the threads when using virtual threads belaban/JGroups#879
Discussion
No response
Motivation
A pinned virtual thread could block progress, especially if the number of CPU cores is small. In #37197 we ensure a thread pool of at least two, still this seems like a workaround until thread pinning is fixed. It might also affect the runtime behavior if you go from 1 to 2 CPU cores.
Details
I'll provide a pull request.
Metadata
Metadata
Assignees
Labels
kind/bugCategorizes a PR related to a bugCategorizes a PR related to a bugpriority/importantMust be worked on very soonMust be worked on very soonrelease/26.0.11release/26.1.3release/26.2.0team/sre