Functions connect-cached and get-connection-pool are not thread-safe

[svetlyak40wt]

Functions connect-cached and get-connection-pool are not thread-safe

[svetlyak40wt]

As a workaround, you may recreate the hash from time to time:

(setf cl-dbi::*threads-connection-pool* (cl-dbi::make-threads-connection-pool))

[fukamachi]

Thank you for reporting.
Adding :synchronized t to make-hash-table might work?

[svetlyak40wt]

As I understand, :synchronized t is SBCL's extensions and will not work on other implementations.

Probably we need a separate system to work this around? Something like trivial-sychronized.?

[svetlyak40wt]

As I see some sort of synchronization was already added:

cl-dbi/src/cache/thread.lisp

Line 37 in 738a74d

(bt:with-lock-held ((cache-pool-lock pool))

And today I've fixed it a little in this pull-request:

#75