Skip to content

Conversation

@guan404ming
Copy link
Member

Related PR

#50443
cc @pierrejeambrun

Why

  • patch_task_instance_note and _patch_task_instance_state in bulk service are very similar to what's in the route for patch_task_instance.

How

  • move patch_task_instance_note and _patch_task_instance_state out of TI bulk service
  • use the new service func to handle patching in the route

^ 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.

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.

Thanks for consolidating the common utility.

@guan404ming
Copy link
Member Author

Thanks!

@jason810496 jason810496 merged commit 3b40323 into apache:main Jun 9, 2025
98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants