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

Wangda Tan commented on YARN-5323:
----------------------------------

Thanks [~curino], 

Some questions / comments: 

1) Who will consume AMRMProxyFederationPolicy and RouterFederationPolicy? I'm 
not be able to figure it out from the patch, I guess AMRMProxyService will use 
them, is it correct?
2) It looks like FederationPolicy is more like a 
FederationConfigurator/Creator, is it better to remove the FederationPolicy 
interface? We can either make FederationPolicyConfigurator extends 
FederationPolicyWriter or let implementation decide which to implement (such as 
{{SimpleFederationPolicyConfiguratorImpl implements FederationPolicyWriter, 
FederationPolicyConfigurator}}).
3) Is it better to rename FederationPolicyContext to 
FederationPolicyInitializationContext? 
4) The name "router" is a little hard to be understood, now it only determines 
application's home sub cluster while submitting the app, do you plan to add 
more functionalities to the module? Do you think is it better to rename it to 
be "AMRMProxyHomeClusterPolicy", (and similarly we may rename 
AMRMProxyFederationPolicy to AMRMProxyResourceRequestPolicy).

> Policies APIs (for Router and AMRMProxy policies)
> -------------------------------------------------
>
>                 Key: YARN-5323
>                 URL: https://issues.apache.org/jira/browse/YARN-5323
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager, resourcemanager
>    Affects Versions: YARN-2915
>            Reporter: Carlo Curino
>            Assignee: Carlo Curino
>         Attachments: YARN-5323-YARN-2915.05.patch, YARN-5323.01.patch, 
> YARN-5323.02.patch, YARN-5323.03.patch, YARN-5323.04.patch
>
>
> This JIRA tracks APIs for the policies that will guide the Router and 
> AMRMProxy decisions on where to fwd the jobs submission/query requests as 
> well as ResourceRequests.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to