kaxil commented on code in PR #45106:
URL: https://github.com/apache/airflow/pull/45106#discussion_r1898676110
##########
airflow/utils/state.py:
##########
@@ -36,9 +36,12 @@ class TerminalTIState(str, Enum):
"""States that a Task Instance can be in that indicate it has reached a
terminal state."""
SUCCESS = "success"
- FAILED = "failed"
+ FAILED = "failed" # This state indicates that we attempt to retry.
SKIPPED = "skipped" # A user can raise a AirflowSkipException from a task
& it will be marked as skipped
REMOVED = "removed"
+ FAIL_WITHOUT_RETRY = (
+ "fail_without_retry" # This state is useful for when we want to
terminate a task, without retrying.
+ )
Review Comment:
```suggestion
FAIL_WITHOUT_RETRY = "fail_without_retry"
```
in this case I think the name is sufficient
--
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]
