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

Giovanni Matteo Fumarola edited comment on YARN-8137 at 4/16/18 10:21 PM:
--------------------------------------------------------------------------

Thanks [~abmodi] for the patch.

Few comments:

1) You have this instruction {{int threadPoolSize = Math.max(poolSize, 10);}} 
instead of 10 you should have {{SLSConfiguration.RUNNER_POOL_SIZE_DEFAULT;}}

{{2)executorService.awaitTermination(1, TimeUnit.HOURS);}} I think the awaiting 
time is too long. How did you come up with this value?


was (Author: giovanni.fumarola):
Thanks [~abmodi] for the patch.

Few comments:

1) You have this instruction {{int threadPoolSize = Math.max(poolSize, 10);}} 
instead of 10 you should have {{SLSConfiguration.RUNNER_POOL_SIZE_DEFAULT;}}

{{2) {{executorService.awaitTermination(1, TimeUnit.HOURS);}}}} I think the 
awaiting time is too long. How did you come up with this value?

> Parallelize node addition in SLS
> --------------------------------
>
>                 Key: YARN-8137
>                 URL: https://issues.apache.org/jira/browse/YARN-8137
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Abhishek Modi
>            Assignee: Abhishek Modi
>            Priority: Major
>         Attachments: YARN-8137.001.patch
>
>
> Right now, nodes are added sequentially and it can take a long time if there 
> are large number of nodes. With this change nodes will be added in parallel 
> and thus reduce the node addition time.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to