ShubhamGondane commented on PR #62910: URL: https://github.com/apache/airflow/pull/62910#issuecomment-4050399413
Thanks for the feedback @pierrejeambrun! I've reworked the approach:
- Moved the explicit .join(dag_run) / .outerjoin(dag_version) into
eager_load_TI_and_TIH_for_validation so all join + eager-loading logic is
centralised in one function
- The function now takes the query as input, adds the joins, applies
contains_eager, and returns the modified query
- Updated all 4: get_task_instances, get_mapped_task_instances,
get_task_instances_batch, and get_task_instance_tries
- Added SQL-structure regression tests for the mapped and batch endpoints
as well
--
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]
