comment out the rollback on after test since pg8000 can deadlock on this (and its...

but then what about the pypy jython team.  ho hum.

diff --git a/test/lib/engines.py b/test/lib/engines.py
index 0c980c376fb2b21c12f5a5fa2bac94611c784c79..6202c0a3df2bddd8b27fcaa1b60ddba04b7f1248 100644 (file)
--- a/test/lib/engines.py
+++ b/test/lib/engines.py
@@ -1,7 +1,7 @@
 import sys, types, weakref
 from collections import deque
 from test.bootstrap import config
-from test.lib.util import decorator
+from test.lib.util import decorator, gc_collect
 from sqlalchemy.util import callable
 from sqlalchemy import event, pool
 from sqlalchemy.engine import base as engine_base
@@ -42,8 +42,13 @@ class ConnectionKiller(object):
                 self._safe(rec._close)
 
     def _after_test_ctx(self):
-        for conn in self.conns:
-            self._safe(conn.rollback)
+        pass
+        # this can cause a deadlock with pg8000 - pg8000 acquires
+        # prepared statment lock inside of rollback() - if async gc
+        # is collecting in finalize_fairy, deadlock.
+        # not sure if this should be if pypy/jython only
+        #for conn in self.conns:
+        #    self._safe(conn.rollback)
 
     def _stop_test_ctx(self):
         self.close_all()