luciano/next-jdbc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Note: broken transaction machinery

Browse source 
After removing records and relying entirely on protocols, transaction handling is broken because you can't `assoc` into a non-map.
This commit is contained in:
Sean Corfield 2019-01-10 18:24:55 -08:00
parent a5027066f9
commit 73a7b2af10
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/next/jdbc.clj
View file

@ -93,6 +93,7 @@
;; should check isolation level; maybe implement save points? ;; should check isolation level; maybe implement save points?
(f con) (f con)
(with-open [^AutoCloseable t-con (assoc (get-connection con) (with-open [^AutoCloseable t-con (assoc (get-connection con)
;; FIXME: not a record/map!
:transacted (atom committable?))] :transacted (atom committable?))]
(let [^Connection jdbc t-con (let [^Connection jdbc t-con
old-autocommit (.getAutoCommit jdbc) old-autocommit (.getAutoCommit jdbc)