From 6ad71cc29d8e0f8f5cc7976a84fa0b953edf0f82 Mon Sep 17 00:00:00 2001 From: Richard van der Hoff Date: Wed, 8 Mar 2017 18:00:44 +0000 Subject: [PATCH] Remove spurious SQL logging (#1972) looks like the upsert function was accidentally sending sql logging to the general logger. We already log the sql in `txn.execute`. --- synapse/storage/_base.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/synapse/storage/_base.py b/synapse/storage/_base.py index 4410cd9e62..a7a8ec9b7b 100644 --- a/synapse/storage/_base.py +++ b/synapse/storage/_base.py @@ -488,10 +488,6 @@ class SQLBaseStore(object): " AND ".join("%s = ?" % (k,) for k in keyvalues) ) sqlargs = values.values() + keyvalues.values() - logger.debug( - "[SQL] %s Args=%s", - sql, sqlargs, - ) txn.execute(sql, sqlargs) if txn.rowcount == 0: @@ -506,10 +502,6 @@ class SQLBaseStore(object): ", ".join(k for k in allvalues), ", ".join("?" for _ in allvalues) ) - logger.debug( - "[SQL] %s Args=%s", - sql, keyvalues.values(), - ) txn.execute(sql, allvalues.values()) return True