[IMP] Improve the speed - Use the inspect module only if we specify the debug log level

bzr revid: stephane@tinyerp.com-20090119224840-j0yqyaw30tzfrc78

1 files changed, 14 insertions, 12 deletions

diff --git a/bin/sql_db.py b/bin/sql_db.py
index 3628213f307..118f8e900dc 100644
--- a/bin/sql_db.py
+++ b/bin/sql_db.py
@@ -82,21 +82,23 @@ class Cursor(object):
         self._obj = self._cnx.cursor(cursor_factory=psycopg1cursor)
         self.autocommit(False)
         self.dbname = pool.dbname
-        
-        from inspect import stack
-        self.__caller = tuple(stack()[2][1:3])
+
+        if tools.config['log_level'] in (netsvc.LOG_DEBUG, netsvc.LOG_DEBUG_RPC):
+            from inspect import stack
+            self.__caller = tuple(stack()[2][1:3])
         
     def __del__(self):
         if hasattr(self, '_obj'):
-            # Oops. 'self' has not been closed explicitly.
-            # The cursor will be deleted by the garbage collector, 
-            # but the database connection is not put back into the connection
-            # pool, preventing some operation on the database like dropping it.
-            # This can also lead to a server overload.
-            msg = "Cursor not closed explicitly\n"  \
-                  "Cursor was created at %s:%s" % self.__caller
-
-            log(msg, netsvc.LOG_WARNING)
+            if tools.config['log_level'] in (netsvc.LOG_DEBUG, netsvc.LOG_DEBUG_RPC):
+                # Oops. 'self' has not been closed explicitly.
+                # The cursor will be deleted by the garbage collector, 
+                # but the database connection is not put back into the connection
+                # pool, preventing some operation on the database like dropping it.
+                # This can also lead to a server overload.
+                msg = "Cursor not closed explicitly\n"  \
+                      "Cursor was created at %s:%s" % self.__caller
+
+                log(msg, netsvc.LOG_WARNING)
             self.close()
 
     @check