jscheffl commented on code in PR #45038:
URL: https://github.com/apache/airflow/pull/45038#discussion_r1895008954
##########
providers/src/airflow/providers/celery/provider.yaml:
##########
@@ -330,6 +330,17 @@ config:
type: string
example: ~
default: "False"
+ extra_celery_config:
+ description: |
+ Extra celery configs to include in the celery worker.
+ Any of the celery config can be added to this config and it
+ will be applied while starting the celery worker. e.g.
{"worker_max_tasks_per_child": 10}
+ See also:
+
https://docs.celeryq.dev/en/stable/userguide/configuration.html#configuration-and-defaults
+ version_added: ~
+ type: string
+ example: ~
+ default: "{}"
Review Comment:
Okay, I also did not know but defaults seem to be Format-String formatted
and the string "{}" is an invalid sequence. So to effectively have an `{}` as
default we need to use `{{}}` here to quote the brackets.
```suggestion
default: "{{}}"
```
--
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]
