[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269720#comment-14269720 ] Zhijie Shen commented on YARN-2880: --- Is https://builds.apache.org/job/PreCommit-YARN-Bui

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269430#comment-14269430 ] Hudson commented on YARN-2880: -- FAILURE: Integrated in Hadoop-Mapreduce-trunk #2018 (See [htt

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269415#comment-14269415 ] Hudson commented on YARN-2880: -- FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #68 (See

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269380#comment-14269380 ] Hudson commented on YARN-2880: -- SUCCESS: Integrated in Hadoop-Hdfs-trunk-Java8 #64 (See [http

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269357#comment-14269357 ] Hudson commented on YARN-2880: -- SUCCESS: Integrated in Hadoop-Hdfs-trunk #1999 (See [https://

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269248#comment-14269248 ] Hudson commented on YARN-2880: -- SUCCESS: Integrated in Hadoop-Yarn-trunk #801 (See [https://b

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269221#comment-14269221 ] Hudson commented on YARN-2880: -- FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #67 (See [http

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

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

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14233729#comment-14233729 ] Hadoop QA commented on YARN-2880: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14232588#comment-14232588 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14232525#comment-14232525 ] Rohith commented on YARN-2880: -- Attached updated patch. Kindly review > Add a test in TestRMR

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14232492#comment-14232492 ] Rohith commented on YARN-2880: -- That is good catch. I will updated test for these 2 operations

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14232171#comment-14232171 ] Jian He commented on YARN-2880: --- actually, we can test remove label and replace label too, c

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14232155#comment-14232155 ] Jian He commented on YARN-2880: --- +1 > Add a test in TestRMRestart to make sure node labels w

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14232120#comment-14232120 ] Hadoop QA commented on YARN-2880: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14231205#comment-14231205 ] Rohith commented on YARN-2880: -- These test failures are unrelated to patch. > Add a test in T

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14231186#comment-14231186 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14231185#comment-14231185 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14231113#comment-14231113 ] Rohith commented on YARN-2880: -- Attached patch that fixes test failures. > Add a test in Tes

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14231089#comment-14231089 ] Rohith commented on YARN-2880: -- The reason for test failures are newly added test was setting

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14230861#comment-14230861 ] Rohith commented on YARN-2880: -- I am looking into test failures > Add a test in TestRMRestart

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14230838#comment-14230838 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14230423#comment-14230423 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14230283#comment-14230283 ] Jian He commented on YARN-2880: --- looks good, re-kick jenkins > Add a test in TestRMRestart t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14226525#comment-14226525 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14226395#comment-14226395 ] Rohith commented on YARN-2880: -- Attached updated patch. I changed static sleep to wait for 5 s

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14225660#comment-14225660 ] Rohith commented on YARN-2880: -- Thanks Jian He for review.. I will update rebased patch fixing

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14225409#comment-14225409 ] Jian He commented on YARN-2880: --- patch looks good. Instead of hard sleep, we can wait until

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14224798#comment-14224798 ] Hadoop QA commented on YARN-2880: - {color:red}-1 overall{color}. Here are the results of t

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14224778#comment-14224778 ] Rohith commented on YARN-2880: -- I update the patch for adding test NodeManagerLabel recovery d

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14222454#comment-14222454 ] Rohith commented on YARN-2880: -- Thanks [~leftnoteasy] for your inputs. I try to upload patch a

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14221271#comment-14221271 ] Wangda Tan commented on YARN-2880: -- [~rohithsharma], Thanks for taking up this JIRA, bq. I

[jira] [Commented] (YARN-2880) Add a test in TestRMRestart to make sure node labels will be recovered if it is enabled

[ https://issues.apache.org/jira/browse/YARN-2880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14219224#comment-14219224 ] Rohith commented on YARN-2880: -- Hi Wangda Tan, I am trying to write test cases for nod