[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-12-13 Thread Junping Du (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15746349#comment-15746349 ] Junping Du commented on YARN-4438: -- Per discussion in YARN-5709, I have merge the patch (apply clean

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-11 Thread Akira AJISAKA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15093181#comment-15093181 ] Akira AJISAKA commented on YARN-4438: - Thanks Steve! > Implement RM leader election with curator >

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15089661#comment-15089661 ] Jian He commented on YARN-4438: --- [~ajisakaa], I think it's complaining that the newly added class requires a

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15089770#comment-15089770 ] Steve Loughran commented on YARN-4438: -- Everyone: yes, the build fails in javadocs. But I don't think

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15089746#comment-15089746 ] Steve Loughran commented on YARN-4438: -- Ignore everything I'm saying ... I'm confused now, and if

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15089700#comment-15089700 ] Jian He commented on YARN-4438: --- [~ajisakaa], actually, what I said is about checkstyle. For the javadoc -1.

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15090235#comment-15090235 ] Jian He commented on YARN-4438: --- no problem, Steve, thanks for spending time on investigating this ! >

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15090074#comment-15090074 ] Steve Loughran commented on YARN-4438: -- OK. I've got a on-line patch YARN-4567 covering the javadocs.

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15089697#comment-15089697 ] Steve Loughran commented on YARN-4438: -- It's complaining that the javadocs are invalid: Java 8 javadoc

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-08 Thread Akira AJISAKA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15089512#comment-15089512 ] Akira AJISAKA commented on YARN-4438: - Hi [~xgong] and [~jianhe], bq. -1 javadoc please do not ignore

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

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

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-07 Thread Xuan Gong (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088220#comment-15088220 ] Xuan Gong commented on YARN-4438: - +1 lgtm. Checking this in. > Implement RM leader election with curator

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-07 Thread Xuan Gong (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088285#comment-15088285 ] Xuan Gong commented on YARN-4438: - Committed into trunk/branch-2. Thanks, Jian. And thanks for the review,

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2016-01-07 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088306#comment-15088306 ] Hudson commented on YARN-4438: -- FAILURE: Integrated in Hadoop-trunk-Commit #9071 (See

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

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

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

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

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-27 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15072436#comment-15072436 ] Jian He commented on YARN-4438: --- bq. Not sure I understand why ZKRMStateStore needs to be an AlwaysOn

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-22 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15069202#comment-15069202 ] Karthik Kambatla commented on YARN-4438: bq. And because ZKRMStateStore is currently in active

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-16 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061126#comment-15061126 ] Jian He commented on YARN-4438: --- Thanks for detailed review ! bq. All that is needed is store the

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-15 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059248#comment-15059248 ] Karthik Kambatla commented on YARN-4438: bq. I'll make the change accordingly. Not sure I

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

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

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

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

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-11 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15053955#comment-15053955 ] Jian He commented on YARN-4438: --- [~ozawa], thanks for reviewing the patch, I fixed your comments; bq. why did

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-10 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15050391#comment-15050391 ] Karthik Kambatla commented on YARN-4438: Would very much like for us to use Curator for leader

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

2015-12-10 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-4438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15052001#comment-15052001 ] Jian He commented on YARN-4438: --- bq. Otherwise, the clients (and their individual timeouts etc.) could lead

[jira] [Commented] (YARN-4438) Implement RM leader election with curator

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