You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "Taragolis (via GitHub)" <gi...@apache.org> on 2023/02/18 05:31:06 UTC

[GitHub] [airflow] Taragolis commented on pull request #29602: Lyso change config max tis per query

Taragolis commented on PR #29602:
URL: https://github.com/apache/airflow/pull/29602#issuecomment-1435489931

   Thank for your contributions.
   
   Could you describe a bit more of configurations of Airflow, your DB backend, how far it away out of Airflow services (latency), and your workload. I asked it because for me it is looks like by this PR we might replace [one magic number](https://github.com/apache/airflow/pull/3324#discussion_r187540176) by another.
   
   I've also found that `max_tis_per_query` use in BackfillJob:
   
   https://github.com/apache/airflow/blob/aa4858d85a00759a4a84bdd5fb3fe6cec196831e/airflow/jobs/backfill_job.py#L957-L959
   
   Do you know how it affects this part of code?
   
   
   
   


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