[Zodb-checkins] SVN: ZODB/branches/jim-thready-zeo2/src/ZEO/zrpc/connection.py Removed debug logging. It's waaay too expensive. There has to be a

Jim Fulton jim at zope.com
Fri Sep 18 15:22:18 EDT 2009


Log message for revision 104357:
  Removed debug logging. It's waaay too expensive.  There has to be a
  better way to handle that.
  

Changed:
  U   ZODB/branches/jim-thready-zeo2/src/ZEO/zrpc/connection.py

-=-
Modified: ZODB/branches/jim-thready-zeo2/src/ZEO/zrpc/connection.py
===================================================================
--- ZODB/branches/jim-thready-zeo2/src/ZEO/zrpc/connection.py	2009-09-18 16:56:33 UTC (rev 104356)
+++ ZODB/branches/jim-thready-zeo2/src/ZEO/zrpc/connection.py	2009-09-18 19:22:18 UTC (rev 104357)
@@ -528,20 +528,12 @@
         # result in asycnore calling handle_error(), which will
         # close the connection.
         msgid, flags, name, args = self.marshal.decode(message)
-
-        if __debug__:
-            self.log("recv msg: %s, %s, %s, %s" % (msgid, flags, name,
-                                                   short_repr(args)),
-                     level=TRACE)
         if name == REPLY:
             self.handle_reply(msgid, flags, args)
         else:
             self.handle_request(msgid, flags, name, args)
 
     def handle_reply(self, msgid, flags, args):
-        if __debug__:
-            self.log("recv reply: %s, %s, %s"
-                     % (msgid, flags, short_repr(args)), level=TRACE)
         self.replies_cond.acquire()
         try:
             self.replies[msgid] = flags, args
@@ -554,17 +546,10 @@
 
         if name.startswith('_') or not hasattr(obj, name):
             if obj is None:
-                if __debug__:
-                    self.log("no object calling %s%s"
-                             % (name, short_repr(args)),
-                             level=logging.DEBUG)
                 return
 
             msg = "Invalid method name: %s on %s" % (name, repr(obj))
             raise ZRPCError(msg)
-        if __debug__:
-            self.log("calling %s%s" % (name, short_repr(args)),
-                     level=logging.DEBUG)
 
         meth = getattr(obj, name)
         try:
@@ -587,9 +572,6 @@
                 raise ZRPCError("async method %s returned value %s" %
                                 (name, short_repr(ret)))
         else:
-            if __debug__:
-                self.log("%s returns %s" % (name, short_repr(ret)),
-                         logging.DEBUG)
             if isinstance(ret, Delay):
                 ret.set_sender(msgid, self.send_reply, self.return_error)
             else:
@@ -673,17 +655,11 @@
     def __call_message(self, method, args, flags):
         # compute a message and return it
         msgid = self.__new_msgid()
-        if __debug__:
-            self.log("send msg: %d, %d, %s, ..." % (msgid, flags, method),
-                     level=TRACE)
         return self.marshal.encode(msgid, flags, method, args)
 
     def send_call(self, method, args, flags):
         # send a message and return its msgid
         msgid = self.__new_msgid()
-        if __debug__:
-            self.log("send msg: %d, %d, %s, ..." % (msgid, flags, method),
-                     level=TRACE)
         buf = self.marshal.encode(msgid, flags, method, args)
         self.message_output(buf)
         return msgid
@@ -750,8 +726,6 @@
 
     def wait(self, msgid):
         """Invoke asyncore mainloop and wait for reply."""
-        if __debug__:
-            self.log("wait(%d)" % msgid, level=TRACE)
 
         self.trigger.pull_trigger()
 
@@ -767,9 +741,6 @@
                 reply = self.replies.get(msgid)
                 if reply is not None:
                     del self.replies[msgid]
-                    if __debug__:
-                        self.log("wait(%d): reply=%s" %
-                                 (msgid, short_repr(reply)), level=TRACE)
                     return reply
                 self.replies_cond.wait()
         finally:
@@ -777,15 +748,11 @@
 
     def flush(self):
         """Invoke poll() until the output buffer is empty."""
-        if __debug__:
-            self.log("flush")
         while self.writable():
             self.poll()
 
     def poll(self):
         """Invoke asyncore mainloop to get pending message out."""
-        if __debug__:
-            self.log("poll()", level=TRACE)
         self.trigger.pull_trigger()
 
 



More information about the Zodb-checkins mailing list