diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2011-03-16 18:15:27 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2011-03-16 18:15:27 -0400 |
| commit | 852a1aef0763b5cd5ffb196ec1d18dcfee6bbfd9 (patch) | |
| tree | 7b751e82e01ef3fbf077d872c5c474cf37cf3e22 /lib/sqlalchemy/pool.py | |
| parent | 798339430a7657023188364c3ead0bcce56abd6b (diff) | |
| download | sqlalchemy-852a1aef0763b5cd5ffb196ec1d18dcfee6bbfd9.tar.gz | |
- Fixed AssertionPool regression bug. A replacement of finalize
logic did the wrong thing, but then also the logic it replaced also
doesn't seem like its needed. If it is, would rather have a test case
first so its out for now, added tests for assertionpool. [ticket:2097]
Diffstat (limited to 'lib/sqlalchemy/pool.py')
| -rw-r--r-- | lib/sqlalchemy/pool.py | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/lib/sqlalchemy/pool.py b/lib/sqlalchemy/pool.py index 327e4aa1f..a45c2e80b 100644 --- a/lib/sqlalchemy/pool.py +++ b/lib/sqlalchemy/pool.py @@ -60,8 +60,6 @@ def clear_managers(): class Pool(log.Identified): """Abstract base class for connection pools.""" - _no_finalize = False - def __init__(self, creator, recycle=-1, echo=None, use_threadlocal=False, @@ -324,9 +322,6 @@ class _ConnectionRecord(object): def _finalize_fairy(connection, connection_record, pool, ref, echo): _refs.discard(connection_record) - if pool._no_finalize: - return - if ref is not None and \ connection_record.fairy is not ref: return @@ -826,8 +821,6 @@ class AssertionPool(Pool): than desired. """ - _no_finalize = True - def __init__(self, *args, **kw): self._conn = None self._checked_out = False |