Let logging format messages on demand, use warning instead of deprecated warn

diff --git a/lib/sqlalchemy/events.py b/lib/sqlalchemy/events.py
index c679db37da1bcec75dfbef59410df34b01ca1dd2..e99c5b5e5afaeec53e7ca182421080d08d48ff3e 100644 (file)
--- a/lib/sqlalchemy/events.py
+++ b/lib/sqlalchemy/events.py
@@ -464,7 +464,7 @@ class ConnectionEvents(event.Events):
 
         def before_cursor_execute(conn, cursor, statement, parameters, context,
                                                         executemany):
-            log.info("Received statement: %s" % statement)
+            log.info("Received statement: %s", statement)
 
         engine = create_engine('postgresql://scott:tiger@localhost/test')
         event.listen(engine, "before_cursor_execute", before_cursor_execute)
@@ -475,7 +475,7 @@ class ConnectionEvents(event.Events):
             @event.listens_for(conn, 'before_cursor_execute')
             def before_cursor_execute(conn, cursor, statement, parameters,
                                             context, executemany):
-                log.info("Received statement: %s" % statement)
+                log.info("Received statement: %s", statement)
 
     When the methods are called with a `statement` parameter, such as in
     :meth:`.after_cursor_execute`, :meth:`.before_cursor_execute` and

diff --git a/lib/sqlalchemy/testing/provision.py b/lib/sqlalchemy/testing/provision.py
index b3812cf0488dc2fd4f479caef654a34247f8bcee..65a3c10a6e2f24a26f345af76accb25461852547 100644 (file)
--- a/lib/sqlalchemy/testing/provision.py
+++ b/lib/sqlalchemy/testing/provision.py
@@ -251,10 +251,10 @@ def _oracle_configure_follower(config, ident):
 def _ora_drop_ignore(conn, dbname):
     try:
         conn.execute("drop user %s cascade" % dbname)
-        log.info("Reaped db: %s" % dbname)
+        log.info("Reaped db: %s", dbname)
         return True
     except exc.DatabaseError as err:
-        log.warn("couldn't drop db: %s" % err)
+        log.warning("couldn't drop db: %s", err)
         return False