use NullPool for SQLite by default for non-:memory: connections ?

Issue #1921 resolved
Mike Bayer repo owner created an issue

it shouldn't be too hard these days to just switch off of a ":memory:" connection or not and just use NullPool for file-based connections. Its a little dated for us to still be doing threadlocal-by-default here.

Comments (4)

  1. Mike Bayer reporter

    attached patch is complete. Also removes the usage of "threadlocal" pool with the "threadlocal" engine strategy (not really needed).

  2. Mike Bayer reporter

    put a note in the SQLite docs also about the "pool_threadlocal" flag, which will help a lot of folks with locking in a single thread.

  3. Log in to comment