dstandish commented on PR #53429:
URL: https://github.com/apache/airflow/pull/53429#issuecomment-3084964690
> There are similar endpoints perhaps that could be reused or updated to add
extra fields?
>
> Get latest run (I cannot find it being used in UI code) Endpoint :
`ui/grid/latest_run/{dag_id}`
>
> Get runs and limit by 1 to get latest run with minimal fields
`GridRunsResponse` like `DAGRunLightResponse` Endpoint :
`ui/grid/runs/{dag_id}?limit=1&order_by=-run_after`
The first one is to find if there's a new run that we haven't seen yet to
start refreshing.
The second one involves sometimes a dag deserialization which is one of the
expensive parts of this.
Maybe in `ui/grid/runs` we could get rid of the conditional deser and reuse
it.
But personally, I am not really sure it's worth the trade off -- coupling in
exchange for fewer endpoints. Like, what's the problem with adding an endpoint
for this?
--
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]
