Remove synchronized works to make it Virtual threads(Loom) friendly

[nitin1532]

There are lot of synchronized keyword, and its getting it Pinned when used with virtual threads enabled. Can you please take the enhancement request to make it Loom friendly in next release.
Currently I am seeing multiple onPinned() events for QueryExecutorImpl.java and method closeStatementAndPortal(). but i could see a lot more places it needs to be removed.

Thanks
Nitin Singhal

[bhvkshah]

Hi @nitin1532 there are several high priority tasks the team is working on currently,. I will add this enhancement request to our backlog but I cannot yet give an ETA. Of course, you are welcome to submit a Pull Request which I will be happy to review.

[nitin1532]

Can you guide me the process for it? Thanks. Regards, Nitin Singhal

…

On Fri, 16 Feb 2024 at 2:17 pm, Bhavik Shah ***@***.***> wrote: Hi @nitin1532 <https://github.com/nitin1532> there are several high priority tasks the team is working on currently,. I will add this enhancement request to our backlog but I cannot yet give an ETA. Of course, you are welcome to submit a Pull Request which I will be happy to review. — Reply to this email directly, view it on GitHub <#108 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKTJXVCHPNDIOTF6WQKQ74DYT5TADAVCNFSM6AAAAABDLV3HDWVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4DIOJTGE3DM> . You are receiving this because you were mentioned.Message ID: <aws/amazon-redshift-jdbc-driver/repo-discussions/108/comments/8493166@ github.com>

[bhvkshah]

Gladly, you can follow the guide here:https://docs.github.com/en/get-started/exploring-projects-on-github/contributing-to-a-project on creating your own branch by forking from the master branch, committing your code changes, and submitting a PR for your changes.