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

Karthik Kambatla commented on YARN-1028:
----------------------------------------

bq. It is unfortunate we are are cloning most of common's 
ConfiguredFailoverProxyProvider
True. However, it is technically HDFS. Abstracting out the logic without the 
HDFS pieces to common and then using those pieces in HDFS and YARN can prove to 
be more work than this way. Also, the files themselves are relatively small ~ 
150 lines. Further, the proxies (RMProxy, ClientRMProxy etc.) are similar but 
not identical. I am afraid it would be more work (and potentially more code to 
maintain) to iron out all these minor issues out.



> Add FailoverProxyProvider like capability to RMProxy
> ----------------------------------------------------
>
>                 Key: YARN-1028
>                 URL: https://issues.apache.org/jira/browse/YARN-1028
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Bikas Saha
>            Assignee: Karthik Kambatla
>         Attachments: yarn-1028-1.patch, yarn-1028-2.patch, yarn-1028-3.patch, 
> yarn-1028-4.patch, yarn-1028-5.patch, yarn-1028-6.patch, yarn-1028-7.patch, 
> yarn-1028-8.patch, yarn-1028-draft-cumulative.patch
>
>
> RMProxy layer currently abstracts RM discovery and implements it by looking 
> up service information from configuration. Motivated by HDFS and using 
> existing classes from Common, we can add failover proxy providers that may 
> provide RM discovery in extensible ways.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to