[ 
https://issues.apache.org/jira/browse/AIRFLOW-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16650256#comment-16650256
 ] 

ASF GitHub Bot commented on AIRFLOW-1945:
-----------------------------------------

phani8996 opened a new pull request #3989: [AIRFLOW-1945] Autoscale celery 
workers for airflow added
URL: https://github.com/apache/incubator-airflow/pull/3989
 
 
   Dear Airflow Maintainers,
   
   This will add a provision to autoscale celery workers unlike same numbers of 
workers irrespective of number of running tasks.
   
   Please accept this PR that addresses the following issues:
   https://issues.apache.org/jira/browse/AIRFLOW-1945
   
   Testing Done:
   
   Manually tested by passing arguments in cli

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Pass --autoscale to celery workers
> ----------------------------------
>
>                 Key: AIRFLOW-1945
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-1945
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: celery, cli
>            Reporter: Michael O.
>            Assignee: Sai Phanindhra
>            Priority: Trivial
>              Labels: easyfix
>   Original Estimate: 0.5h
>  Remaining Estimate: 0.5h
>
> Celery supports autoscaling of the worker pool size (number of tasks that can 
> parallelize within one worker node).  I'd like to propose to support passing 
> the --autoscale parameter to {{airflow worker}}.
> Since this is a trivial change, I am not sure if there's any reason for not 
> being supported already.(?)
> For example
> {{airflow worker --concurrency=4}} will set a fixed pool size of 4.
> With minimal changes in 
> [https://github.com/apache/incubator-airflow/blob/4ce4faaeae7a76d97defcf9a9d3304ac9d78b9bd/airflow/bin/cli.py#L855]
>  it could support
> {{airflow worker --autoscale=2,10}} to set an autoscaled pool size of 2 to 10
> Some references:
> * 
> http://docs.celeryproject.org/en/latest/internals/reference/celery.worker.autoscale.html
> * 
> https://github.com/apache/incubator-airflow/blob/4ce4faaeae7a76d97defcf9a9d3304ac9d78b9bd/airflow/bin/cli.py#L855



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to