arjav1528 commented on PR #60161: URL: https://github.com/apache/airflow/pull/60161#issuecomment-3761699291
> > Thank you for that pull request. People have been asking for this feature back a lot! 🎉 > > For the two reasons mentioned bellow (crafting the payload requires all the TIs of the group which isn't easy to get on the client side because of pagination + the return type of the API will create confusion). > > I think the best approach is to create a couple of endpoint specifically for patching a TI group. (dry_run and non dry_run version). Refactoring and reusing the code of the `patch` ti endpoint, because it already returns a `TaskInstanceCollectionResponse` (for the scenario of patching all mapped index at once). > > This should make this PR much simpler. (basically just a simple call to patch, with the group id + payload, and apply the change to all tasks within the group) > > This feature is also personally on my bucket list! I would refactor this PR inline with @pierrejeambrun 's suggestions Thanks a lot, also would be happy to recieve some suggestions -- 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]
