dstandish commented on code in PR #23317:
URL: https://github.com/apache/airflow/pull/23317#discussion_r896048129


##########
tests/always/test_example_dags.py:
##########
@@ -60,7 +60,7 @@ def test_should_be_importable(example):
 
 @pytest.mark.parametrize("example", example_dags_except_db_exception(), 
ids=relative_path)
 def test_should_not_do_database_queries(example):
-    with assert_queries_count(0):
+    with assert_queries_count(0, 
ignore_patterns=['dagbag.py:validate_task_pools']):

Review Comment:
   hmm moving this to dag processor seems like generally the right direction 
but i think i may need to rethink the sync logic because we won't always have 
all warnings in memory, since the processor may just run for a specific dag 
file and not the entire dags folder...



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