From 420f20df24b90c1f1598369ef2c45a904fa9ecc8 Mon Sep 17 00:00:00 2001 From: Klaas van Schelven Date: Fri, 18 Apr 2025 14:36:57 +0200 Subject: [PATCH] Transaction-level logging: clarify 'using' --- bugsink/transaction.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/bugsink/transaction.py b/bugsink/transaction.py index bbf4f42..23fda45 100644 --- a/bugsink/transaction.py +++ b/bugsink/transaction.py @@ -52,7 +52,8 @@ class SemaphoreContext: inc_stat(self.using, "get_write_lock", took) # textually, slightly misleading since it's not literally "BEGIN IMMEDIATE" we're waiting for here (instead: the # semaphore) but it's clear enough - performance_logger.info(f"{took * 1000:6.2f}ms BEGIN IMMEDIATE, A.K.A. get-write-lock") + using_clause = f" ({ self.using })" if self.using != DEFAULT_DB_ALIAS else "" + performance_logger.info(f"{took * 1000:6.2f}ms BEGIN IMMEDIATE, A.K.A. get-write-lock{using_clause}") def __exit__(self, exc_type, exc_value, traceback): immediate_semaphores[self.using].release() @@ -178,7 +179,8 @@ class ImmediateAtomic(SuperDurableAtomic): took = time.time() - self.t0 inc_stat(self.using, "immediate_transaction", took) - performance_logger.info(f"{took * 1000:6.2f}ms IMMEDIATE transaction") + using_clause = f" ({ self.using })" if self.using != DEFAULT_DB_ALIAS else "" + performance_logger.info(f"{took * 1000:6.2f}ms IMMEDIATE transaction{using_clause}") connection = django_db_transaction.get_connection(self.using) if hasattr(connection, "_start_transaction_under_autocommit"):