tjhub1983 commented on PR #66218:
URL: https://github.com/apache/airflow/pull/66218#issuecomment-4380619481
@parkhojeong @bugraoz93 All review suggestions have been addressed:
**Changes made:**
1. ****: Removed wrapper class; now directly calls and — consistent with
the existing pattern from the API
2. ** **: Changed signature to accept pydantic model instead of individual
parameters — follows the same pattern as and other operations classes
3. ** parsing**: Added to parse comma-separated CLI argument into list
4. ****: Updated to use instead of keyword args
```python
# Before
ops = TaskOperations(client=api_client.client)
cleared = ops.clear(dag_id=args.dag_id, only_failed=True,
task_ids=args.task_ids, ...)
# After
body = ClearTaskInstancesBody(only_failed=True, task_ids=task_ids.split(",")
if task_ids else None, ...)
cleared = api_client.client.tasks.clear(dag_id=args.dag_id, body=body)
```
--
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]
