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

Karthik Kambatla commented on YARN-4559:
----------------------------------------

Quickly skimmed through the patch. High-level comments:
# Curator start and close should be in the same class. Let us do that both in 
the RM or both in the elector service. My preference would be for doing it in 
the RM. Is there any reason that wouldn't work? 
# The ZK-store specific configs - adding the RM username:password for 
exclusivity should likely be done in the serviceInit of the store. 

> Make leader elector and zk store share the same curator client
> --------------------------------------------------------------
>
>                 Key: YARN-4559
>                 URL: https://issues.apache.org/jira/browse/YARN-4559
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jian He
>            Assignee: Jian He
>         Attachments: YARN-4559.1.patch
>
>
> After YARN-4438, we can reuse the same curator client for leader elector and 
> zk store



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

Reply via email to