docs: Add some comments

lablup · Oct 4, 2021 · 1259b3a · 1259b3a
1 parent 9e0cd14
commit 1259b3a
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/ai/backend/manager/models/utils.py b/src/ai/backend/manager/models/utils.py
@@ -79,6 +79,9 @@ async def begin_readonly(self, deferrable: bool = False) -> AsyncIterator[SAConn
     @actxmgr
     async def advisory_lock(self, lock_id: AdvisoryLock) -> AsyncIterator[None]:
         lock_acquired = False
+        # Here we use the session-level advisory lock,
+        # which follows the lifetime of underlying DB connection.
+        # As such, we should keep using one single connection for both lock and unlock ops.
         async with self.connect() as lock_conn:
             try:
                 # It is usually a BAD practice to directly interpolate strings into SQL statements,