fat-catTW commented on PR #71654:
URL: https://github.com/apache/airflow/pull/71654#issuecomment-5311565436

   Thanks for the review.
   
   I updated the guard to key off the actual filtered attribute instead of only 
the query parameter name, so `dag_run_start_date` now gets the same 
null-start-date handling as `start_date`.
   
   I also kept `dag_run_end_date` on the plain range filter path to avoid 
changing the existing `/dags` end-date behavior, and added `DagRun.id IS NOT 
NULL` to the start-date null guard so the outer join does not accidentally 
include Dags with no latest run.
   
   Added coverage for the `/dags` case with a latest Dag run that has 
`start_date=NULL` and a populated `end_date`.
   
   Verified locally with the focused regression tests, the existing `/dags` 
filter matrix, and `mypy-airflow-core`.


-- 
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]

Reply via email to