guan404ming commented on code in PR #71073:
URL: https://github.com/apache/airflow/pull/71073#discussion_r3763323928
##########
providers/common/ai/src/airflow/providers/common/ai/operators/llm_branch.py:
##########
@@ -69,11 +77,13 @@ def __init__(
self,
*,
allow_multiple_branches: bool = False,
+ fail_on_reject: bool = False,
Review Comment:
Agreed on keeping the family consistent. Moving the flag to LLMApprovalMixin
needs skip machinery (inherits_from_skipmixin, teardown-aware skip) on the
non-branch operators plus a decision on their default, so I'd rather do it as a
follow-up PR to keep this one tight. Will open it after this merges.
--
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]
