Do away with pool._refs

This collection was added only for the benefit of unit tests
and is unnecessary for the pool to function.  As SQLAlchemy 2.0
will be removing the automatic handling of connections that are
garbage collection, remove this collection so that we ultimately
don't need a weakref handler to do anything within the pool.
The handler will do nothing other than emit a warning that
a connection was dereferenced without being explicitly returned
to the pool, invalidated, or detached.

Change-Id: I4ca196270d5714efbac44dbf6f034e8c7f0af58a

1 files changed, 0 insertions, 8 deletions

diff --git a/test/aaa_profiling/test_pool.py b/test/aaa_profiling/test_pool.py
index d74dcf627..538eee45d 100644
--- a/test/aaa_profiling/test_pool.py
+++ b/test/aaa_profiling/test_pool.py
@@ -1,4 +1,3 @@
-from sqlalchemy import pool as pool_module
 from sqlalchemy.pool import QueuePool
 from sqlalchemy.testing import AssertsExecutionResults
 from sqlalchemy.testing import fixtures
@@ -18,13 +17,6 @@ class QueuePoolTest(fixtures.TestBase, AssertsExecutionResults):
         def close(self):
             pass
 
-    def teardown(self):
-        # the tests leave some fake connections
-        # around which don't necessarily
-        # get gc'ed as quickly as we'd like all the time,
-        # particularly for non-refcount gc
-        pool_module._refs.clear()
-
     def setup(self):
         # create a throwaway pool which
         # has the effect of initializing