sunank200 commented on PR #51880:
URL: https://github.com/apache/airflow/pull/51880#issuecomment-2987329491

   > Users expect it to return tasks whose state IS NULL.
   
   The change currently there `return [None if s in ("no_status", "none", None) 
else TaskInstanceState(s) for s in states].` is needed.
   
   But one more change is needed.  In both `get_task_instances` and 
`get_mapped_task_instances` methods, if the incoming `state` filter includes 
`None`, we now *omit* all the date-range filters (run_after, start_date, etc.), 
since tasks with no state have no dates and would otherwise be filtered out. We 
only apply the non-date filters (state, pool, queue, executor, etc.) in that 
case.


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to