eladkal commented on code in PR #43520:
URL: https://github.com/apache/airflow/pull/43520#discussion_r1841650852
##########
airflow/config_templates/config.yml:
##########
@@ -2331,6 +2331,14 @@ scheduler:
type: integer
example: ~
default: "8974"
+ num_stuck_in_queued_retries:
Review Comment:
I am late to join the party but I don't follow why we need this. We have
[AIRFLOW__SCHEDULER__TASK_QUEUED_TIMEOUT](https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#task-queued-timeout)
which introduced in Airflow 2.6
Which means that task retries after timeout passed. If for some reason this
setting isn't doing it now the we should fix it rather than adding a new
setting.
`task_queued_timeout` + `num_stuck_in_queued_retries` feels too complicated.
--
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]
