search

IBM / JTOpen

IBM Toolbox for Java, an IBM i communications library
https://ibm.github.io/JTOpen/
Other
60 stars 28 forks source link

Loom initial work #125

Closed mdg1349 closed 9 months ago

mdg1349 commented 1 year ago

In this branch, I used explicit locks instead of the keyword synchronized to make the codebase more friendly towards JDK 21 and Loom. I did not replace every single synchronized block, just the ones that were shown using the option -Djdk.tracePinnedThreads=full in a simple program that creates some connections from an AS400JDBCDataSource using setThreadUsed(false) and then executes a somewhat simple select inside a virtual thread

Comments appreciated!

ThePrez commented 11 months ago

@mdg1349, thank you for your contributions! We have not yet made the time to fully understand the implications of loom, so this one will sit "simmering" for a bit still. This is important work and we appreciate what you've started here.

ThePrez commented 10 months ago

Just to update, we're taking a look at these proposed changes and hope to get it merged soon!

ThePrez commented 9 months ago

Thank you!!

mdg1349 commented 9 months ago

Thank you! I'm eagerly waiting for the next release!!