tjhub1983 commented on PR #66218:
URL: https://github.com/apache/airflow/pull/66218#issuecomment-4380739047
All review feedback addressed:
1. **ClearTaskInstancesBody** - TaskOperations.clear() now accepts
ClearTaskInstancesBody as the single parameter (not individual kwargs) -
matches the pattern from AIP-94 PR #63889
2. **api_client.tasks.clear()** - task_command.py now calls
api_client.client.tasks.clear() directly instead of wrapping in TaskOperations
class
3. **task_ids parsing** - Added task_ids.split(",") to parse the
comma-separated CLI argument
4. **help text** - Both commands have clear help text for "Clear task
instances" and "Get task instance states for a Dag run"
5. **Test updated** - test_operations.py test_clear now uses
ClearTaskInstancesBody(only_failed=True)
Fixed in commits on airflow-tasks-cli branch.
--
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]
