[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-19 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13535979#comment-13535979 ] Hudson commented on YARN-230: - Integrated in Hadoop-Mapreduce-trunk #1290 (See [https://builds.

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-19 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13535937#comment-13535937 ] Hudson commented on YARN-230: - Integrated in Hadoop-Hdfs-trunk #1259 (See [https://builds.apach

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-19 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13535863#comment-13535863 ] Hudson commented on YARN-230: - Integrated in Hadoop-Yarn-trunk #70 (See [https://builds.apache.

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13535634#comment-13535634 ] Hudson commented on YARN-230: - Integrated in Hadoop-trunk-Commit #3140 (See [https://builds.apa

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-17 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13534098#comment-13534098 ] Bikas Saha commented on YARN-230: - Thanks guys! > Make changes for RM resta

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-17 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13534044#comment-13534044 ] Tom White commented on YARN-230: Arun, yes it looks good to me, +1. We can address any chang

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-15 Thread Arun C Murthy (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13533094#comment-13533094 ] Arun C Murthy commented on YARN-230: Tom, I think I'm comfortable with the patch though

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-14 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13532597#comment-13532597 ] Bikas Saha commented on YARN-230: - A note on testing. Among other test changes, there is a f

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-14 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13532499#comment-13532499 ] Bikas Saha commented on YARN-230: - Yes. Will do when I refresh the patch on YARN-231 after Y

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-14 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13532371#comment-13532371 ] Tom White commented on YARN-230: OK, in which case please make the default state store the f

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-13 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13531255#comment-13531255 ] Bikas Saha commented on YARN-230: - I think the NullRMStateStore is a coding convenience as o

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-13 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13531147#comment-13531147 ] Tom White commented on YARN-230: Thanks for addressing my feedback Bikas. The NullRMStateSto

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-12 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13530439#comment-13530439 ] Bikas Saha commented on YARN-230: - Thanks for the reviews guys! @Tom Sorry for the delayed

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-12 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13530413#comment-13530413 ] Hadoop QA commented on YARN-230: {color:green}+1 overall{color}. Here are the results of te

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-12-11 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13529601#comment-13529601 ] Hadoop QA commented on YARN-230: {color:red}-1 overall{color}. Here are the results of test

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13507475#comment-13507475 ] Hadoop QA commented on YARN-230: {color:green}+1 overall{color}. Here are the results of te

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13507444#comment-13507444 ] Hadoop QA commented on YARN-230: {color:red}-1 overall{color}. Here are the results of test

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13507361#comment-13507361 ] Hadoop QA commented on YARN-230: {color:red}-1 overall{color}. Here are the results of test

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13505567#comment-13505567 ] Hadoop QA commented on YARN-230: {color:red}-1 overall{color}. Here are the results of test

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-28 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13505427#comment-13505427 ] Tom White commented on YARN-230: bq. But it could make sense to remove application attempts

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-27 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13504775#comment-13504775 ] Bikas Saha commented on YARN-230: - bq. Why is this flexibility needed? I can't see why it ma

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-27 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13504765#comment-13504765 ] Tom White commented on YARN-230: {quote} The store and remove methods have been made mirrors

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-24 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13503371#comment-13503371 ] Bikas Saha commented on YARN-230: - Ignoring orphaned attempts was explicitly added because o

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-23 Thread Tom White (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13503228#comment-13503228 ] Tom White commented on YARN-230: Overall, this looks great. General feedback: * Can we make

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-20 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13501361#comment-13501361 ] Bikas Saha commented on YARN-230: - Attaching 4 patches that break up the whole change for ea

[jira] [Commented] (YARN-230) Make changes for RM restart phase 1

2012-11-20 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13501213#comment-13501213 ] Bikas Saha commented on YARN-230: - Attached patch applies on top of patch for YARN-229