pierrejeambrun commented on issue #59598: URL: https://github.com/apache/airflow/issues/59598#issuecomment-3773304467
@nclaeys Yes, we forgot to remove the `pending-response` label, hence the auto closing. Sorry for that. Indeed the issue you mention still occurs, that's caused by the Tasks cards trying to load past TI execution. Compared to the Dags card that uses a specific endpoint fetching all at once and that is paginated. First of all the `get_tasks` endpoint should probably be paginated. We don't want 1 api call returning 5000 tasks if the dag is huge. THat would limit the number of parallel call in the UI. I think we need to convert the UI table into a simple 'table' not 'card'. If we really want card, we need an extra ui endpoint similar to get_dags `ui` that will recent 'recent_runs' as well in the endpoint. So 1 call -> 1 page of tasks with 14 recent execution. > I saw in the issues that work has been done to fix n+1 queries (https://github.com/apache/airflow/issues/57561), but it seems this path has been missed. I also just want to mention that this is about another problem (hence the confusion). It fixed how the backend would perform N+1 db queries for a single API endpoint call. The issue reported here is how the UI would make multiple calls to the backend to retrieve information. -- 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]
