[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15765667#comment-15765667 ] Karthik Kambatla commented on YARN-4559: Just pulled this back into branch-2.8 as well. > Make

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15109425#comment-15109425 ] Jian He commented on YARN-4559: --- the test failures are not related. > Make leader elector and zk store

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15109683#comment-15109683 ] Hudson commented on YARN-4559: -- FAILURE: Integrated in Hadoop-trunk-Commit #9143 (See

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15109635#comment-15109635 ] Xuan Gong commented on YARN-4559: - +1 lgtm. checking this in > Make leader elector and zk store share the

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15109646#comment-15109646 ] Xuan Gong commented on YARN-4559: - Committed into trunk/branch-2. Thanks, Jian > Make leader elector and

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15106516#comment-15106516 ] Hadoop QA commented on YARN-4559: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15106212#comment-15106212 ] Xuan Gong commented on YARN-4559: - The last patch looks good to me. Looks like the test cases failures

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15101786#comment-15101786 ] Hadoop QA commented on YARN-4559: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15102978#comment-15102978 ] Hadoop QA commented on YARN-4559: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15101162#comment-15101162 ] Hadoop QA commented on YARN-4559: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15099154#comment-15099154 ] Karthik Kambatla commented on YARN-4559: bq. I think the best way is just to merge the

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15099163#comment-15099163 ] Jian He commented on YARN-4559: --- bq. ZK-leader election with a different store. I think currently only

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15099185#comment-15099185 ] Karthik Kambatla commented on YARN-4559: There "could" be other stores that provide the same

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15099068#comment-15099068 ] Jian He commented on YARN-4559: --- new patch moved the curator#close in RM#serviceStop. I cannot move the

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15095598#comment-15095598 ] Xuan Gong commented on YARN-4559: - Thanks, Jian for working on this, and Thanks, Karithik for the review.

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088403#comment-15088403 ] Karthik Kambatla commented on YARN-4559: Quickly skimmed through the patch. High-level comments: #

[jira] [Commented] (YARN-4559) Make leader elector and zk store share the same curator client

[ https://issues.apache.org/jira/browse/YARN-4559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088616#comment-15088616 ] Jian He commented on YARN-4559: --- bq. My preference would be for doing it in the RM yep, I can move the close