[GitHub] [airflow] coolbeans201 commented on a change in pull request #17317: docs: fix inconsistencies in configuration docs

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

coolbeans201 commented on a change in pull request #17317:
URL: https://github.com/apache/airflow/pull/17317#discussion_r679271491



##########
File path: airflow/config_templates/config.yml
##########
@@ -1831,36 +1831,30 @@
       default: "True"
     - name: max_dagruns_to_create_per_loop
       description: |
-        Max number of DAGs to create DagRuns for per scheduler loop
-
-        Default: 10
+        Max number of DAGs to create DagRuns for per scheduler loop.
       example: ~
       version_added: 2.0.0
       type: string
-      default: ~
+      default: "10"
       see_also: ":ref:`scheduler:ha:tunables`"
     - name: max_dagruns_per_loop_to_schedule
       description: |
         How many DagRuns should a scheduler examine (and lock) when scheduling
         and queuing tasks.
-
-        Default: 20
       example: ~
       version_added: 2.0.0
       type: string
-      default: ~
+      default: "20"
       see_also: ":ref:`scheduler:ha:tunables`"
     - name: schedule_after_task_execution
       description: |
         Should the Task supervisor process perform a "mini scheduler" to attempt to schedule more tasks of the
         same DAG. Leaving this on will mean tasks in the same DAG execute quicker, but might starve out other
         dags in some circumstances
-
-        Default: True
       example: ~
       version_added: 2.0.0
       type: boolean
-      default: ~
+      default: "True"

Review comment:
       Shouldn't this be True (without quotations) since it's a boolean?




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