Skip to content

Commit

Permalink
Fix duplicate backfill CANCELING status check (dagster-io#23077)
Browse files Browse the repository at this point in the history
Summary:
From the comment it looks like this was supposed to be CANCELED? There
is already a CANCELING check above, so this code would never run.

Test Plan: BK

## Summary & Motivation

## How I Tested These Changes
  • Loading branch information
gibsondan authored Jul 23, 2024
1 parent 68f9636 commit 8a7bdd8
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1125,7 +1125,7 @@ def execute_asset_backfill_iteration(
logger.debug(
f"Updated asset backfill data after cancellation iteration: {updated_asset_backfill_data}"
)
elif backfill.status == BulkActionStatus.CANCELING:
elif backfill.status == BulkActionStatus.CANCELED:
# The backfill was forcibly canceled, skip iteration
pass
else:
Expand Down

0 comments on commit 8a7bdd8

Please sign in to comment.