[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15072436#comment-15072436 ]
Jian He commented on YARN-4438: ------------------------------- bq. Not sure I understand why ZKRMStateStore needs to be an AlwaysOn service. It does not need to be always on, just the zkClient in ZKRMStateStore needs to be always on. bq. How would this change look? At first glance, in AdminService#transitionToStandby and transitionToActive, not call refreshAll if the shared-storage-config-provider is not enabled. bq. Is the concern that Curator may be biased in picking an RM in certain conditions? Yeah, that's just my guess. Immediately rejonning may have more chance to take leadership again. ActiveStandbyElector#reJoinElectionAfterFailureToBecomeActive has similar comments. bq.If leaderLatch.close() throws an exception, when does Curator realize the RM is not participating in the election anymore? Based on my understanding, I think curator will realize when it does not hear RM for the zkSessionTimeout period. Essentially, the zkClient at RM side will keep retrying to notify zk quorum that this client is closed. If close successds, zk quorum will get notified immediately and re-selects a leader. If close is kept retrying for beyond zkSessionTimeout, zk quorum will assume this client dies and re-selects a leader. bq. we might not need that thread. Then, we can remove this thread ? I'll do separately if you agree. bq. What happens if this RM is subsequently elected leader? Does the transition to Active succeed just fine? I think it can transition to active next time it's selected as leader. The previous failure will most likely happen on refreshAcl. bq. I would like for us to err on the side of caution and do null-checks. will do > Implement RM leader election with curator > ----------------------------------------- > > Key: YARN-4438 > URL: https://issues.apache.org/jira/browse/YARN-4438 > Project: Hadoop YARN > Issue Type: Improvement > Reporter: Jian He > Assignee: Jian He > Attachments: YARN-4438.1.patch, YARN-4438.2.patch, YARN-4438.3.patch > > > This is to implement the leader election with curator instead of the > ActiveStandbyElector from common package, this also avoids adding more > configs in common to suit RM's own needs. -- This message was sent by Atlassian JIRA (v6.3.4#6332)