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

Carlo Curino commented on YARN-5324:
------------------------------------

[~yochen] thanks for reviewing this.  

 # The {{ProbabilisticRouterFederationPolicy}} was renamed to 
{{WeightedRandomRouterFederationPolicy}}. 
 # The snippet you quote was still there, good catch. I added to all test cases 
a corner check that forces us to be aware of weights missing info about an 
active clusters, and fixed the NPE in {{WeightedRandomRouterFederationPolicy}} 
and other classes. 

Also: 
 # The latest patch also includes a {{LoadBasedRouterFederationPolicy}} that 
parses the {{ClusterMetricsInfo}} from String capability and picks the cluster 
with the most free space {{max(availableMemoryMB}} (and adds tests for that and 
{{OrderedRouterFederationPolicy}}.
 # We could probably polish the tests (I think there is a bunch of redundancy. 





> Stateless router policies implementation
> ----------------------------------------
>
>                 Key: YARN-5324
>                 URL: https://issues.apache.org/jira/browse/YARN-5324
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager, resourcemanager
>            Reporter: Carlo Curino
>            Assignee: Carlo Curino
>         Attachments: YARN-5324.01.patch, YARN-5324.02.patch, 
> YARN-5324.03.patch, YARN-5324.04.patch
>
>
> These are policies at the Router that do not require maintaing state across 
> choices (e.g., weighted random).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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