dstandish commented on code in PR #60216:
URL: https://github.com/apache/airflow/pull/60216#discussion_r2759573329
##########
airflow-core/tests/unit/models/test_backfill.py:
##########
@@ -363,18 +368,54 @@ def test_active_dag_run(dag_maker, session):
dag_run_conf={"this": "param"},
)
assert b1 is not None
- with pytest.raises(AlreadyRunningBackfill, match="Another backfill is
running for dag"):
+
+ # Try to create overlapping backfill - should fail
+ with pytest.raises(AlreadyRunningBackfill, match="Another backfill is
running for Dag"):
Review Comment:
So when the backfill is actually being created, then for each run in the
range, it will check "is this part of an active backfill?" If it is, it will
just skip that date.
--
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]
