[Checkins] SVN: ZODB/branches/3.7/src/ZEO/zrpc/connection.py Removed a needless timeout to a condition wait call. Using timeouts

Marius Gedminas marius at pov.lt
Sun Apr 1 15:48:42 EDT 2007


Log message for revision 73967:
  Removed a needless timeout to a condition wait call. Using timeouts
  can cause signidficant delays, especially on systems with very
  course-grained sleeps, like most linux systems.
  
  Extracted from rev 73871 in svn://svn.zope.org/repos/main/ZODB/trunk/src/ZEO
  (Jim Fulton did the extracting and handed me the diff on IRC).
  
  

Changed:
  U   ZODB/branches/3.7/src/ZEO/zrpc/connection.py

-=-
Modified: ZODB/branches/3.7/src/ZEO/zrpc/connection.py
===================================================================
--- ZODB/branches/3.7/src/ZEO/zrpc/connection.py	2007-04-01 19:37:41 UTC (rev 73966)
+++ ZODB/branches/3.7/src/ZEO/zrpc/connection.py	2007-04-01 19:48:42 UTC (rev 73967)
@@ -439,6 +439,9 @@
         self.closed = True
         self.__super_close()
         self.close_trigger()
+        self.replies_cond.acquire()
+        self.replies_cond.notifyAll()
+        self.replies_cond.release()
 
     def close_trigger(self):
         # Overridden by ManagedClientConnection.
@@ -734,7 +737,7 @@
                                  (msgid, short_repr(reply)), level=TRACE)
                     return reply
                 if self.is_async():
-                    self.replies_cond.wait(10.0)
+                    self.replies_cond.wait()
                 else:
                     self.replies_cond.release()
                     try:



More information about the Checkins mailing list