You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/02/28 14:06:58 UTC

[GitHub] [airflow] kaxil commented on issue #7587: [AIRFLOW-6957] Make retrieving Paused Dag ids a separate method

kaxil commented on issue #7587: [AIRFLOW-6957] Make retrieving Paused Dag ids a separate method
URL: https://github.com/apache/airflow/pull/7587#issuecomment-592527753
 
 
   > Can you also add a test that checks if the DagFileProcessor ignores these DAGs? This is the core, and it's worth one more test than not having it.
   
   We already have other cases covered:
   https://github.com/apache/airflow/blob/847cb40a9087d7aa515744d4553d09b7c63c0e2a/tests/jobs/test_scheduler_job.py#L847-L854
   https://github.com/apache/airflow/blob/847cb40a9087d7aa515744d4553d09b7c63c0e2a/tests/jobs/test_scheduler_job.py##L1162-L1188
   
   The bug in the last PR was we were not checking how the results were returned from the query which this PR adds.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services