Skip to content

Conversation

@Lee-W
Copy link
Member

@Lee-W Lee-W commented Aug 5, 2025

Why

It would be better if we had a way to invoke the notifier right after the HITLDetail is created.

What

Add an optional notifier argument to HITLOperator which leverage the existing Airflow notifier


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@Lee-W Lee-W moved this to In review in AIP-90 - Human in the loop Aug 5, 2025
@Lee-W Lee-W changed the title feat(htil): add notifiers argument to HITLOperator feat(htil): add notifiers to HITLOperator Aug 5, 2025
@Lee-W Lee-W requested review from jason810496 and jscheffl August 5, 2025 11:34
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, very cool!

Copy link
Member

@jason810496 jason810496 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Nice feature!

@Lee-W Lee-W merged commit 9b1c16c into apache:main Aug 6, 2025
105 checks passed
@Lee-W Lee-W deleted the add-notifier-to-hitl branch August 6, 2025 01:54
@github-project-automation github-project-automation bot moved this from In review to Done in AIP-90 - Human in the loop Aug 6, 2025
ferruzzi pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Aug 7, 2025
fweilun pushed a commit to fweilun/airflow that referenced this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants