search

seancorfield / next-jdbc

A modern low-level Clojure wrapper for JDBC-based access to databases.
https://cljdoc.org/d/com.github.seancorfield/next.jdbc/
Eclipse Public License 1.0
750 stars 89 forks source link

clojure.java.jdbc compatibility in concurrent connection usage #245

Closed seancorfield closed 1 year ago

seancorfield commented 1 year ago

Is your feature request related to a problem? Please describe. See #244 for background.

Describe the solution you'd like If you bind *nested-tx* to :ignore to get c.j.j compatibility there, you should also get compatibility on the (non-)locking of Connection to match c.j.j's unsafe behavior (see #106 for why that changed in the first place).

seancorfield commented 1 year ago

@camsaul this is on develop and in the latest 1.3.999-SNAPSHOT for testing.

camsaul commented 1 year ago

Thanks, I'll give it a shot tomorrow!

camsaul commented 1 year ago

Thanks @seancorfield, the fix works for us! 🥳