jason810496 commented on code in PR #60161:
URL: https://github.com/apache/airflow/pull/60161#discussion_r2724224917
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/task_instances.py:
##########
@@ -921,11 +921,17 @@ def bulk_task_instances(
dag_bag: DagBagDep,
dag_run_id: str,
user: GetUserDep,
-) -> BulkResponse:
+ dry_run: bool = Query(
+ False, description="If True, return affected task instances without
making changes"
+ ),
+) -> BulkResponse | TaskInstanceCollectionResponse:
Review Comment:
The problem hasn't been solved yet, we need to separate the routes for
dry_run and non-dry_run one.
##########
.github/workflows/run-unit-tests.yml:
##########
@@ -175,7 +175,8 @@ jobs:
AIRFLOW_MONITOR_DELAY_TIME_IN_SECONDS:
"${{inputs.monitor-delay-time-in-seconds}}"
VERBOSE: "true"
DEFAULT_BRANCH: "${{ inputs.default-branch }}"
- TOTAL_TEST_TIMEOUT: "3600" # 60 minutes in seconds
+ # Default is 60 minutes; Min SQLAlchemy providers DB runs can exceed
that slightly.
+ TOTAL_TEST_TIMEOUT: "${{ (inputs.test-group == 'providers' &&
inputs.downgrade-sqlalchemy == 'true') && '3900' || '3600' }}"
if: inputs.test-group == 'core' || inputs.skip-providers-tests != 'true'
Review Comment:
I thinks this is a bad rebase. The change doesn't seem to be related to the
PR.
--
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]
