tirkarthi commented on issue #23419:
URL: https://github.com/apache/airflow/issues/23419#issuecomment-1116106610

   @bbovenzi I would like to pick this up in case you are not working on this. 
I have a rough working version of this and can also work on queue, run 
endpoints. I would like to confirm below things.
   
   1. The linked API has `confirmed` parameter which will return a `str` value 
of task_instances on false. Do we need to support this parameter in query 
parameter? If so do we need to return `str` value of task_instances or do we 
need to return like the endpoint where task_instances for a dag_run or returned.
   2. Since we have `dag_id` and `dag_run_id` in the URL do we need this as a 
post endpoint since confirmed is also a query parameter and we are not posting 
additional data whereas the linked API uses post since `/dagrun_clear` has no 
details of `dag_id` and `dag_run_id`. It makes sense for post if the API will 
be expanded in future to support more customisation.


-- 
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]

Reply via email to