[GitHub] [airflow] ashb commented on a change in pull request #21546: Allow to switch easily between Bullseye and Buster debian versions

[GitBox <gi...@apache.org>]

ashb commented on a change in pull request #21546:
URL: https://github.com/apache/airflow/pull/21546#discussion_r815201542



##########
File path: tests/jobs/test_scheduler_job.py
##########
@@ -3902,13 +3902,16 @@ def test_process_dags_queries_count(
 
             failures = []  # Collects assertion errors and report all of them at the end.
             message = "Expected {expected_count} query, but got {current_count} located at:"
-            for expected_query_count in expected_query_counts:
-                with create_session() as session:
-                    try:
-                        with assert_queries_count(expected_query_count, message):
-                            self.scheduler_job._do_scheduling(session)
-                    except AssertionError as e:
-                        failures.append(str(e))
+            # in Debian Bullseye for MSSQL one of the tests gets into additional loop
+            # More often, and we should increase the interval of fetching to avoid that
+            with conf_vars({('core', 'min_serialized_dag_fetch_interval'): '30'}):

Review comment:
       On L3889 We set the value to 100 (seconds), so this setting here actually _decreases_ the timeout, so this comment doesn't quite hold up.




-- 
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-unsubscribe@airflow.apache.org

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