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

Young Chen commented on YARN-7708:
----------------------------------

Thanks [~botong] for the comments. I've fixed all of your suggestions except 
for moving the properties to the YarnConfiguration. Since this 
LoadBasedGlobalPolicy is pluggable, and may not be in use at times, is it still 
considered good practice to put the properties inside YarnConfiguration? I feel 
that this might lead to excessive bloat in the already very large configuration 
file.

In addition, I also changed scaling to be an enum. I think this should be 
cleaner than arbitrary strings.

 

[^YARN-7708-YARN-7402.03.patch]

> [GPG] Load based policy generator
> ---------------------------------
>
>                 Key: YARN-7708
>                 URL: https://issues.apache.org/jira/browse/YARN-7708
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Carlo Curino
>            Assignee: Young Chen
>            Priority: Major
>         Attachments: YARN-7708-YARN-7402.01.cumulative.patch, 
> YARN-7708-YARN-7402.01.patch, YARN-7708-YARN-7402.02.cumulative.patch, 
> YARN-7708-YARN-7402.02.patch, YARN-7708-YARN-7402.03.cumulative.patch, 
> YARN-7708-YARN-7402.03.patch, YARN-7708-YARN-7402.04.cumulative.patch, 
> YARN-7708-YARN-7402.05.cumulative.patch, 
> YARN-7708-YARN-7402.06.cumulative.patch, 
> YARN-7708-YARN-7402.07.cumulative.patch
>
>
> This policy reads load from the "pendingQueueLength" metrics and provides 
> scaling into a set of weights that influence the AMRMProxy and Router 
> behaviors.



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