o-nikolas commented on code in PR #32572:
URL: https://github.com/apache/airflow/pull/32572#discussion_r1262829467
##########
airflow/config_templates/config.yml:
##########
@@ -2451,14 +2451,15 @@ scheduler:
max_tis_per_query:
description: |
This changes the batch size of queries in the scheduling main loop.
+ This should not be greater than ``core.parallelism``.
If this is too high, SQL query performance may be impacted by
complexity of query predicate, and/or excessive locking.
Additionally, you may hit the maximum allowable query length for your
db.
- Set this to 0 for no limit (not advised)
+ Setting this to 0 for using the value of ``core.parallelism``
Review Comment:
> I would say _to use_
You'd need to change "Setting" --> "Set" if you use that language, which
does change the meaning a bit more than intended I think. But both options are
better than what was there before :+1:
--
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]
