You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/11 13:17:40 UTC

[GitHub] [airflow] potiuk commented on a change in pull request #13612: Change the default celery worker_concurrency to 16

potiuk commented on a change in pull request #13612:
URL: https://github.com/apache/airflow/pull/13612#discussion_r555038693



##########
File path: UPDATING.md
##########
@@ -52,6 +52,13 @@ assists users migrating to a new version.
 
 ## Master
 
+### Default `[celery] worker_concurrency` is changed to `16`

Review comment:
       Can we also add env variable to set the ''8" value in `entrypoint_ci,sh` - the number of celery workers also impacts the tests run in GitHub so it would be great to keep the lower value there to keep stability.

##########
File path: UPDATING.md
##########
@@ -52,6 +52,13 @@ assists users migrating to a new version.
 
 ## Master
 
+### Default `[celery] worker_concurrency` is changed to `16`

Review comment:
       Can we also add env variable to set the ''8" value in `entrypoint_ci.sh` - the number of celery workers also impacts the tests run in GitHub so it would be great to keep the lower value there to keep stability.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org