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/10/10 02:15:12 UTC

[GitHub] [airflow] jhtimmins commented on pull request #11189: Prepend `DAG:` to dag permissions

jhtimmins commented on pull request #11189:
URL: https://github.com/apache/airflow/pull/11189#issuecomment-706469601


   @ashb @kaxil If y'all can take a look. As long as tests are passing, you can merge if you don't have any issues.
   
   The new support for multiple schedulers seems to cause variance in the number of queries at times, so I added `margin_of_error` to the `assert_queries_count` function. The default is 0, but for some `test_scheduler_job.py` I set it to `2`. So if the expected number of queries is 95, tests will pass with 93-97 queries. This should solve variance in number of queries due to test/DB flakiness.


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