Yoann Rodière (
https://hibernate.atlassian.net/secure/ViewProfile.jspa?accountId=557058%...
) *created* an issue
Hibernate Search (
https://hibernate.atlassian.net/browse/HSEARCH?atlOrigin=eyJpIjoiZmU3OTY0...
) / Improvement (
https://hibernate.atlassian.net/browse/HSEARCH-4712?atlOrigin=eyJpIjoiZmU...
) HSEARCH-4712 (
https://hibernate.atlassian.net/browse/HSEARCH-4712?atlOrigin=eyJpIjoiZmU...
) Convert synchronized blocks to java.util.concurrent.Locks for better Loom compatibility
(
https://hibernate.atlassian.net/browse/HSEARCH-4712?atlOrigin=eyJpIjoiZmU...
)
Issue Type: Improvement Assignee: Unassigned Created: 05/Oct/2022 08:08 AM Fix Versions:
6.2-backlog Priority: Major Reporter: Yoann Rodière (
https://hibernate.atlassian.net/secure/ViewProfile.jspa?accountId=557058%...
)
See
https://hibernate.atlassian.net/browse/HHH-15541 (
https://hibernate.atlassian.net/browse/HHH-15541 ) , apparently synchronized blocks cause
problems with Loom by “pinning threads”.
What that means exactly is secondary, I think; the important part is that migrating from
synchronized blocks to locks should be trivial, so we might as well do it.
So, let’s remove synchronized blocks from non-test code, and use locks instead.
Also, while we’re at it, we should probably add a checkstyle rule to forbid synchronized
blocks (and suggest using locks instead, with a link to this ticket).
(
https://hibernate.atlassian.net/browse/HSEARCH-4712#add-comment?atlOrigin...
) Add Comment (
https://hibernate.atlassian.net/browse/HSEARCH-4712#add-comment?atlOrigin...
)
Get Jira notifications on your phone! Download the Jira Cloud app for Android (
https://play.google.com/store/apps/details?id=com.atlassian.android.jira....
) or iOS (
https://itunes.apple.com/app/apple-store/id1006972087?pt=696495&ct=Em...
) This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100207- sha1:427dab2 )