diff options
author | Ville Skyttä <ville.skytta@iki.fi> | 2016-05-03 18:29:41 +0300 |
---|---|---|
committer | Ville Skyttä <ville.skytta@iki.fi> | 2016-05-03 18:29:41 +0300 |
commit | 1322edc7b1effb5bad411742b997cb2103eedfc2 (patch) | |
tree | 6501017f1d7b99de62a033f33108b4988278e0fa | |
parent | bde46e33593805584c7c0dedb3a666909fb67888 (diff) | |
download | sqlalchemy-pr/265.tar.gz |
Let logging format messages on demand, use warning instead of deprecated warnpr/265
-rw-r--r-- | lib/sqlalchemy/events.py | 4 | ||||
-rw-r--r-- | lib/sqlalchemy/testing/provision.py | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/lib/sqlalchemy/events.py b/lib/sqlalchemy/events.py index c679db37d..e99c5b5e5 100644 --- 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 b3812cf04..65a3c10a6 100644 --- 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 |