plexus-python-pulse-common 1.1.85__py3-none-any.whl → 1.1.87__py3-none-any.whl

plexus/pulse/common/utils/ormutils.py

@@ -948,6 +948,14 @@ def make_snapshot_model_trigger[SnapshotModelT: SnapshotModelMixin](engine: sa.E
                 NEW."expired_at" := NULL;
                 NEW."record_sqn" := nextval('{record_sqn_seq_name}');
             ELSE
+                -- Serialize transitions even when there is no active version.
+                PERFORM "sqn" FROM "{table_name}"
+                WHERE "record_sqn" = NEW."record_sqn"
+                ORDER BY "sqn" LIMIT 1 FOR UPDATE;
+                IF NOT FOUND THEN
+                    RAISE EXCEPTION 'record_sqn % does not exist', NEW."record_sqn";
+                END IF;
+
                 IF NEW."created_at" IS NULL THEN
                     NEW."created_at" := CURRENT_TIMESTAMP;
                 END IF;
@@ -1019,6 +1027,14 @@ def make_revision_model_trigger[RevisionModelT: RevisionModelMixin](engine: sa.E
                 NEW."record_sqn" := nextval('{record_sqn_seq_name}');
                 NEW."revision" := 1;
             ELSE
+                -- Serialize transitions even when there is no active version.
+                PERFORM "sqn" FROM "{table_name}"
+                WHERE "record_sqn" = NEW."record_sqn"
+                ORDER BY "sqn" LIMIT 1 FOR UPDATE;
+                IF NOT FOUND THEN
+                    RAISE EXCEPTION 'record_sqn % does not exist', NEW."record_sqn";
+                END IF;
+
                 SELECT MAX("created_at") INTO NEW."created_at"
                 FROM "{table_name}"
                 WHERE "record_sqn" = NEW."record_sqn";
@@ -1070,6 +1086,7 @@ def clone_sequence_model_instance[SequenceModelT: SequenceModelMixin](
         return instance
     if clear_meta_fields:
         result.sqn = None
+        result.model_fields_set.discard("sqn")
     return result
 
 
@@ -1087,6 +1104,9 @@ def clone_changing_model_instance[ChangingModelT: ChangingModelMixin](
         result.sqn = None
         result.created_at = None
         result.updated_at = None
+        result.model_fields_set.discard("sqn")
+        result.model_fields_set.discard("created_at")
+        result.model_fields_set.discard("updated_at")
     return result
 
 
@@ -1105,6 +1125,10 @@ def clone_snapshot_model_instance[SnapshotModelT: SnapshotModelMixin](
         result.created_at = None
         result.expired_at = None
         result.record_sqn = None
+        result.model_fields_set.discard("sqn")
+        result.model_fields_set.discard("created_at")
+        result.model_fields_set.discard("expired_at")
+        result.model_fields_set.discard("record_sqn")
     return result
 
 
@@ -1125,6 +1149,12 @@ def clone_revision_model_instance[RevisionModelT: RevisionModelMixin](
         result.expired_at = None
         result.record_sqn = None
         result.revision = None
+        result.model_fields_set.discard("sqn")
+        result.model_fields_set.discard("created_at")
+        result.model_fields_set.discard("updated_at")
+        result.model_fields_set.discard("expired_at")
+        result.model_fields_set.discard("record_sqn")
+        result.model_fields_set.discard("revision")
     return result
 
 
@@ -1144,6 +1174,41 @@ def db_make_order_by_clause[SequenceModelT: SequenceModelMixin](
     return order_criteria
 
 
+def db_lock_model_record[ModelT: RevisionModelMixin | SnapshotModelMixin](
+    db: sa_orm.Session,
+    model: type[ModelT],
+    record_sqn: int,
+) -> None:
+    """
+    Lock the first history row, including when the record is inactive.
+
+    Select by ``record_sqn`` rather than assuming it equals the first ``sqn``: trigger-managed
+    records use a separate sequence. Every lifecycle writer must acquire this same lock before
+    reading current state.
+    Locks last until caller commit or rollback. Do not delete the first history row.
+    """
+    db_anchor = (
+        db
+        .query(model)
+        .where(model.record_sqn == record_sqn)
+        .order_by(model.sqn)
+        .limit(1)
+        .with_for_update()
+        .populate_existing()
+        .one_or_none()
+    )
+    if db_anchor is None:
+        raise sa_exc.NoResultFound(f"'{model_name_of(model)}' of specified record_sqn '{record_sqn}' not found")
+
+
+# Mutating helpers flush but never begin, commit, or roll back transactions. Callers must roll
+# back on failure. Row locks serialize cooperating writers on databases supporting SELECT FOR UPDATE;
+# SQLite callers must serialize writes themselves (e.g. begin an IMMEDIATE transaction before
+# reading).
+# At stronger isolation levels, callers must also handle serialization failures by retrying
+# transactions. History rows used as lock anchors must not be deleted while a record exists.
+
+
 def db_create_sequence_model[SequenceModelT: SequenceModelMixin](
     db: sa_orm.Session,
     model: type[SequenceModelT],
@@ -1201,11 +1266,11 @@ def db_update_sequence_model[SequenceModelT: SequenceModelMixin](
     instance: SequenceModelMixin,
     sqn: int,
 ) -> SequenceModelT:
-    db_instance = db.query(model).where(model.sqn == sqn).one_or_none()
+    db_instance = db.query(model).where(model.sqn == sqn).with_for_update().populate_existing().one_or_none()
     if db_instance is None:
         raise sa_exc.NoResultFound(f"'{model_name_of(model)}' of specified sqn '{sqn}' not found")
 
-    db_instance = model_copy_from(db_instance, clone_sequence_model_instance(model, instance), exclude_none=True)
+    db_instance = model_copy_from(db_instance, clone_sequence_model_instance(model, instance), exclude_unset=True)
     clone_sequence_model_instance(model, db_instance, validate_only=True)
     db.flush()
 
@@ -1217,7 +1282,7 @@ def db_delete_sequence_model[SequenceModelT: SequenceModelMixin](
     model: type[SequenceModelT],
     sqn: int,
 ) -> SequenceModelT:
-    db_instance = db.query(model).where(model.sqn == sqn).one_or_none()
+    db_instance = db.query(model).where(model.sqn == sqn).with_for_update().populate_existing().one_or_none()
     if db_instance is None:
         raise sa_exc.NoResultFound(f"'{model_name_of(model)}' of specified sqn '{sqn}' not found")
 
@@ -1265,11 +1330,11 @@ def db_update_changing_model[ChangingModelT: ChangingModelMixin](
     sqn: int,
     updated_at: datetime.datetime,
 ) -> ChangingModelT:
-    db_instance = db.query(model).where(model.sqn == sqn).one_or_none()
+    db_instance = db.query(model).where(model.sqn == sqn).with_for_update().populate_existing().one_or_none()
     if db_instance is None:
         raise sa_exc.NoResultFound(f"'{model_name_of(model)}' of specified sqn '{sqn}' not found")
 
-    db_instance = model_copy_from(db_instance, clone_changing_model_instance(model, instance), exclude_none=True)
+    db_instance = model_copy_from(db_instance, clone_changing_model_instance(model, instance), exclude_unset=True)
     db_instance.updated_at = updated_at
     clone_changing_model_instance(model, db_instance, validate_only=True)
     db.flush()
@@ -1425,7 +1490,14 @@ def db_update_snapshot_model[SnapshotModelT: SnapshotModelMixin](
     record_sqn: int,
     updated_at: datetime.datetime,
 ) -> SnapshotModelT:
-    db_instance = db.query(model).where(model.record_sqn == record_sqn, model.expired_at.is_(None)).one_or_none()
+    db_lock_model_record(db, model, record_sqn)
+    db_instance = (
+        db
+        .query(model)
+        .where(model.record_sqn == record_sqn, model.expired_at.is_(None))
+        .populate_existing()
+        .one_or_none()
+    )
     if db_instance is None:
         raise sa_exc.NoResultFound(f"active '{model_name_of(model)}' of specified record_sqn '{record_sqn}' not found")
 
@@ -1450,10 +1522,12 @@ def db_expire_snapshot_model[SnapshotModelT: SnapshotModelMixin](
     record_sqn: int,
     updated_at: datetime.datetime,
 ) -> SnapshotModelT:
+    db_lock_model_record(db, model, record_sqn)
     db_instance = (
         db
         .query(model)
         .where(model.record_sqn == record_sqn, model.expired_at.is_(None))
+        .populate_existing()
         .one_or_none()
     )
     if db_instance is None:
@@ -1472,16 +1546,24 @@ def db_activate_snapshot_model[SnapshotModelT: SnapshotModelMixin](
     record_sqn: int,
     updated_at: datetime.datetime,
 ) -> SnapshotModelT:
-    db_instance = db.query(model).where(model.record_sqn == record_sqn, model.expired_at.is_(None)).one_or_none()
+    db_lock_model_record(db, model, record_sqn)
+    db_instance = (
+        db
+        .query(model)
+        .where(model.record_sqn == record_sqn, model.expired_at.is_(None))
+        .populate_existing()
+        .one_or_none()
+    )
     if db_instance is not None:
         raise sa_exc.MultipleResultsFound(
-            f"Active '{model_name_of(model)}' of specified record_sqn '{record_sqn}' already exists")
+            f"active '{model_name_of(model)}' of specified record_sqn '{record_sqn}' already exists")
 
     db_instance = (
         db
         .query(model)
         .where(model.record_sqn == record_sqn, model.expired_at.is_not(None))
-        .order_by(model.created_at.desc())
+        .order_by(model.created_at.desc(), model.sqn.desc())
+        .populate_existing()
         .first()
     )
     if db_instance is None:
@@ -1653,7 +1735,14 @@ def db_update_revision_model[RevisionModelT: RevisionModelMixin](
     record_sqn: int,
     updated_at: datetime.datetime,
 ) -> RevisionModelT:
-    db_instance = db.query(model).where(model.record_sqn == record_sqn, model.expired_at.is_(None)).one_or_none()
+    db_lock_model_record(db, model, record_sqn)
+    db_instance = (
+        db
+        .query(model)
+        .where(model.record_sqn == record_sqn, model.expired_at.is_(None))
+        .populate_existing()
+        .one_or_none()
+    )
     if db_instance is None:
         raise sa_exc.NoResultFound(f"active '{model_name_of(model)}' of specified record_sqn '{record_sqn}' not found")
 
@@ -1680,10 +1769,12 @@ def db_expire_revision_model[RevisionModelT: RevisionModelMixin](
     record_sqn: int,
     updated_at: datetime.datetime,
 ) -> RevisionModelT:
+    db_lock_model_record(db, model, record_sqn)
     db_instance = (
         db
         .query(model)
         .where(model.record_sqn == record_sqn, model.expired_at.is_(None))
+        .populate_existing()
         .one_or_none()
     )
     if db_instance is None:
@@ -1702,7 +1793,14 @@ def db_activate_revision_model[RevisionModelT: RevisionModelMixin](
     record_sqn: int,
     updated_at: datetime.datetime,
 ) -> RevisionModelT:
-    db_instance = db.query(model).where(model.record_sqn == record_sqn, model.expired_at.is_(None)).one_or_none()
+    db_lock_model_record(db, model, record_sqn)
+    db_instance = (
+        db
+        .query(model)
+        .where(model.record_sqn == record_sqn, model.expired_at.is_(None))
+        .populate_existing()
+        .one_or_none()
+    )
     if db_instance is not None:
         raise sa_exc.MultipleResultsFound(
             f"active '{model_name_of(model)}' of specified record_sqn '{record_sqn}' already exists")
@@ -1712,6 +1810,7 @@ def db_activate_revision_model[RevisionModelT: RevisionModelMixin](
         .query(model)
         .where(model.record_sqn == record_sqn, model.expired_at.is_not(None))
         .order_by(model.revision.desc())
+        .populate_existing()
         .first()
     )
     if db_instance is None:

{plexus_python_pulse_common-1.1.85.dist-info → plexus_python_pulse_common-1.1.87.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: plexus-python-pulse-common
-Version: 1.1.85
+Version: 1.1.87
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13

{plexus_python_pulse_common-1.1.85.dist-info → plexus_python_pulse_common-1.1.87.dist-info}/RECORD

@@ -17,14 +17,14 @@ plexus/pulse/common/utils/dockerutils.py,sha256=7DF9yFGz8uHwpYk115xBWqOjs5etz0UE
 plexus/pulse/common/utils/flowutils.py,sha256=MGu15KL9mw-LyGGLceVmY-c2FGer5PURnMZWFSJN--I,10182
 plexus/pulse/common/utils/gisutils.py,sha256=Cx0ZB9-NA_7VuzbuzRArTJ5UfltG-KmvVC8YxPIgPE4,11600
 plexus/pulse/common/utils/jsonutils.py,sha256=zSbW94klN2jrkMdBECu4RNJX6zzNaSxauQ3FuihuB6E,3342
-plexus/pulse/common/utils/ormutils.py,sha256=FPkkdmiTZ1M0ohLTQakA8shO1MD_2mhM6NolpnD4OWo,64472
+plexus/pulse/common/utils/ormutils.py,sha256=ycs8MF5EDfxx8TP4gDrD7zIunxB0N3hSvGcB4adopkA,68187
 plexus/pulse/common/utils/pathutils.py,sha256=eYfNK48wf1xZGK2IbpKxYOn4skZhWWA8OTyUwDXGMIc,9222
 plexus/pulse/common/utils/s3utils.py,sha256=5q5wSOavt-n5EdrflQX7WPGgDeWQkGO9lG5Nq3QXjCk,35904
 plexus/pulse/common/utils/sqlutils.py,sha256=D6kTBjhO5YlNRt3uFlPt6z3uH61m9ajEzPYmsI6NoFc,231
 plexus/pulse/common/utils/strutils.py,sha256=PkmMHCcHN9vALCy_8fJTf00Dg7I2N4QTCXdBmB3t7rk,16440
 plexus/pulse/common/utils/tagutils.py,sha256=V8eUxtWSijX6nHSCuRX2dTJz3why3TCOzPVK5YXt28s,66944
 plexus/pulse/common/utils/testutils.py,sha256=hKPRvoN4mg8QGQJH121l59a0uo8jN5rntGQAVYplGBw,6166
-plexus_python_pulse_common-1.1.85.dist-info/METADATA,sha256=khRqgCk5BE_g8XkuhmSDVTfVKQhEd_hTr6u9CrRb2-Y,1555
-plexus_python_pulse_common-1.1.85.dist-info/WHEEL,sha256=a-udL2m4V_6Yn8b-uKzWb12RjRxwwMo6ohYFM24tpaY,91
-plexus_python_pulse_common-1.1.85.dist-info/top_level.txt,sha256=ug_g7CVwaMQuas5UzAXbHUrQvKGCn8ezc6ZNvvRlJOE,7
-plexus_python_pulse_common-1.1.85.dist-info/RECORD,,
+plexus_python_pulse_common-1.1.87.dist-info/METADATA,sha256=SnylMrcOPgHpUmVJgX4jAdGVuiBl8KBOsSUlmX2U2xs,1555
+plexus_python_pulse_common-1.1.87.dist-info/WHEEL,sha256=mOdXR9C3wMVov7cSY8PY-XCzo3SmmWCHvGQr5csJoao,91
+plexus_python_pulse_common-1.1.87.dist-info/top_level.txt,sha256=ug_g7CVwaMQuas5UzAXbHUrQvKGCn8ezc6ZNvvRlJOE,7
+plexus_python_pulse_common-1.1.87.dist-info/RECORD,,

{plexus_python_pulse_common-1.1.85.dist-info → plexus_python_pulse_common-1.1.87.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: setuptools (84.2.0)
+Generator: setuptools (84.2.1)
 Root-Is-Purelib: true
 Tag: py3-none-any