[ 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