ashb commented on a change in pull request #19842:
URL: https://github.com/apache/airflow/pull/19842#discussion_r757579779
##########
File path: airflow/utils/db.py
##########
@@ -986,3 +988,47 @@ def check(session=None):
"""
session.execute('select 1 as is_alive;')
log.info("Connection successful.")
+
+
[email protected]
+class DBLocks(enum.IntEnum):
+ """
+ Cross-db Identifiers for advisory global database locks.
+
+ Postgres uses int64 lock ids so we use the integer value, MySQL uses
names, so we use the ``_name_``
+ field.
+ """
+
+ INIT = enum.auto()
+ MIGRATIONS = enum.auto()
Review comment:
Yeah, I wasn't sure. Previously there were _three_ separate locks: one
for upgrade, one for reset, and one for the "DAG sync" part of init.
This init lock could perhaps be dropped entirely, yes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
