dstandish commented on code in PR #46554:
URL: https://github.com/apache/airflow/pull/46554#discussion_r1952682256
##########
tests/models/test_backfill.py:
##########
@@ -271,12 +266,12 @@ def _get_bdr(date):
# 2021-01-04 is "failed" so it may or may not be reprocessed depending
# on the configuration
- bdr = _get_bdr("2021-01-04")
+ bdr = _get_bdr("2021-01-05")
actual_reason = bdr.exception_reason
if reprocess_behavior is ReprocessBehavior.FAILED:
- assert actual_reason == BackfillDagRunExceptionReason.ALREADY_EXISTS
+ assert actual_reason == BackfillDagRunExceptionReason.IN_FLIGHT
elif reprocess_behavior is ReprocessBehavior.COMPLETED:
- assert actual_reason is None
+ assert actual_reason == BackfillDagRunExceptionReason.IN_FLIGHT
Review Comment:
we sholud not add an exception reason if we're clearing an existing run i
think
--
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]
