vemikhaylov commented on a change in pull request #14500:
URL: https://github.com/apache/airflow/pull/14500#discussion_r587250624
##########
File path: airflow/api_connexion/openapi/v1.yaml
##########
@@ -2218,6 +2218,13 @@ components:
type: boolean
default: true
+ task_ids:
+ description: A list of task ids to clear.
+ type: array
+ items:
+ type: string
+ minItems: 1
Review comment:
Yes, actually I have two points there:
1) Empty list for `task_ids` can look a little ambiguous. Should it be
treated as `null` and no filters should be applied in the case? Either should
no tasks be cleared?
2) If no tasks should be cleared in the case of empty list, what is the
purpose to call the endpoint?
What do you think?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
