The GitHub Actions job "Tests AMD" on airflow.git/feature/batch-ti-summary-endpoint has failed. Run started by GitHub user dheerajturaga (triggered by potiuk).
Head commit for run: 9ca152b5a09b26d9de92611cf4cf03802009bac7 / Dheeraj Turaga <[email protected]> perf(ui): Add batch endpoint for grid view task instance summaries This change eliminates the N+1 query problem in the grid view by introducing a batch API endpoint that fetches task instance summaries for multiple DAG runs in a single request. **Backend Changes:** - Add POST /grid/ti_summaries_batch/{dag_id} endpoint that accepts multiple run_ids - Implement get_batch_ti_summaries() service function with optimized single-query approach - Add GridTISummariesBatch data model for batch response - Add validation to limit batch requests to max 100 runs **Frontend Changes:** - Create useGridTISummariesBatch hook that fetches all TI summaries at once - Update Grid component to use batch hook instead of individual fetches - Update Bar component to receive summaries as props rather than fetching independently **Performance Impact:** For a grid showing 50 DAG runs: - HTTP requests: 52 → 3 (94% reduction) - Database queries: 50+ → 1 (98% reduction) - Estimated load time: ~5-10s → ~500ms (90% faster) **Testing:** - Add comprehensive unit tests for batch endpoint - Test validation of empty run_ids and max limit enforcement - Verify correct data structure and state aggregation for multiple runs Report URL: https://github.com/apache/airflow/actions/runs/18624843098 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
