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/03/22 20:40:18 UTC

[GitHub] [airflow] jmcarp commented on a change in pull request #6379: [AIRFLOW-5708] Optionally check task pools when parsing dags.

jmcarp commented on a change in pull request #6379: [AIRFLOW-5708] Optionally check task pools when parsing dags.
URL: https://github.com/apache/airflow/pull/6379#discussion_r396138293
 
 

 ##########
 File path: airflow/config_templates/default_airflow.cfg
 ##########
 @@ -208,6 +208,8 @@ dag_discovery_safe_mode = True
 # The number of retries each task is going to have by default. Can be overridden at dag or task level.
 default_task_retries = 0
 
+check_task_pools = False
 
 Review comment:
   I was thinking it would technically be a breaking change to fail dag parsing on missing pools.  What do you think? cc @ashb @potiuk, since you all discussed this issue in https://github.com/apache/airflow/pull/6451.

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