wuvt / wuvt-site

Website including a CMS, playlist information, and donation management tools
https://www.wuvt.vt.edu
GNU Affero General Public License v3.0
11 stars 12 forks source link

Use SQLAlchemy 1.2's built-in pessimistic disconnect handling #302

Closed mutantmonkey closed 6 years ago

mutantmonkey commented 7 years ago

When SQLAlchemy 1.2 is stable, we should start using the new pessimistic disconnect handling by adding pool_pre_ping=True when creating the engine. This will ensure that we don't end up trying to use stale DB connections.

See http://docs.sqlalchemy.org/en/latest/changelog/migration_12.html#pessimistic-disconnection-detection-added-to-the-connection-pool

mutantmonkey commented 7 years ago

See https://github.com/wuvt/impala/pull/2