shivaam commented on code in PR #64089:
URL: https://github.com/apache/airflow/pull/64089#discussion_r2973067579
##########
airflow-core/src/airflow/api_fastapi/core_api/datamodels/backfills.py:
##########
@@ -54,6 +54,8 @@ class BackfillResponse(BaseModel):
completed_at: datetime | None
updated_at: datetime
dag_display_name: str = Field(validation_alias=AliasPath("dag_model",
"dag_display_name"))
+ num_runs: int = 0
+ dag_run_state_counts: dict[str, int] = {}
Review Comment:
Thanks for the review! I used = {} to match the existing pattern in the same
file Happy to change if you'd prefer a different approach, did you have
something specific in mind?
Also curious about your thoughts on the broader design question: would you
prefer these fields on a separate endpoint (similar to dagStats) rather than
enriching BackfillResponse? I am thinking I should create a new API that lists
all the dagRuns associated with a backfill instead
--
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]
