Transaction isolation applied to running transaction in HikariPool about hikaricp HOT 7 CLOSED

I am committing a fix for the former. The trouble with the later is that the user can alter the isolation level of the connection, just like auto-commit, and therefore it needs to be reset.

from hikaricp.

Fix committed to dev.

from hikaricp.

Thanks.

Regarding the caching thing, I thought of this:

Make getTransactionIsolation really cheap by caching the isolation level in a field in the wrapping Connection.
setTransactionIsolation does its thing and also writes to this field.

getConnection on the pool then can call the now cheap getTransactionIsolation and only call setTransactionIsolation if the default pool level is different, thus saving the roundtrip to the database.

But if that really is worth the added complexity is up to you.

from hikaricp.

It's not necessarily the complexity I worry about, I'm more concerned that there is a reason that PostgreSQL is required to make the roundtrip. Otherwise, why would they have implemented the driver that way?

from hikaricp.

That's a good point. Seems more like an optimization the driver should do if possible. I will check with the pgjdbc team.

from hikaricp.

You may be interested in this thread:

https://groups.google.com/forum/#!topic/hikari-cp/7hgIWvE7DiY

Basically, like PostgreSQL, this user found that SQL Server performs a roundtrip on setTransactionIsolation(). I have checked in an optimization to the dev branch that will only reset the transaction isolation level if it is detected that the user actually altered it. In the base case, this completely avoids the call.

from hikaricp.

Thanks for the info.

Just as a follow up:
The pgjdbc drivers setTransactionIsolation sets the isolation level through SQL. So nothing is preventing a user from altering the isolation level without the setTransactionIsolation-API.
I guess that's the reason why getTransactionIsolation has to roundtrip to the database to be certain. If one wants 100% equally configured connections to be handed out from the pool, I'm not sure if the setTransactionIsolation call can be avoided.

from hikaricp.