lohitkolluri commented on code in PR #67900:
URL: https://github.com/apache/airflow/pull/67900#discussion_r3507295586
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/backfills.py:
##########
@@ -282,7 +303,12 @@ def create_backfill(
@backfills_router.post(
path="/dry_run",
- responses=create_openapi_http_exception_doc([status.HTTP_404_NOT_FOUND,
status.HTTP_409_CONFLICT]),
Review Comment:
Not intentional. The 409 response for create_backfill_dry_run was
accidentally dropped when 503 was added. I've restored it in the route
decorator, OpenAPI spec, and generated TS client so it now matches main (404 +
409) while also including 503. Although _do_dry_run doesn't currently raise
AlreadyRunningBackfill, the 409 is kept for API contract consistency.
--
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]
