[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-10 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14092069#comment-14092069 ] Hudson commented on YARN-1954: -- FAILURE: Integrated in Hadoop-Yarn-trunk #640 (See

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-10 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14092090#comment-14092090 ] Hudson commented on YARN-1954: -- SUCCESS: Integrated in Hadoop-Hdfs-trunk #1833 (See

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-10 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14092099#comment-14092099 ] Hudson commented on YARN-1954: -- SUCCESS: Integrated in Hadoop-Mapreduce-trunk #1859 (See

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-10 Thread Tsuyoshi OZAWA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14092425#comment-14092425 ] Tsuyoshi OZAWA commented on YARN-1954: -- Thank you for your review and comments,

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-09 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14091839#comment-14091839 ] Hadoop QA commented on YARN-1954: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-09 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14091852#comment-14091852 ] Hudson commented on YARN-1954: -- FAILURE: Integrated in Hadoop-trunk-Commit #6042 (See

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-08 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14091447#comment-14091447 ] Zhijie Shen commented on YARN-1954: --- +1 except some nits: 1. The abstract method can

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-08 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14091621#comment-14091621 ] Hadoop QA commented on YARN-1954: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087501#comment-14087501 ] Hadoop QA commented on YARN-1954: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087523#comment-14087523 ] Hadoop QA commented on YARN-1954: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087690#comment-14087690 ] Hadoop QA commented on YARN-1954: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-06 Thread Tsuyoshi OZAWA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14087732#comment-14087732 ] Tsuyoshi OZAWA commented on YARN-1954: -- It's ready for review. [~zjshen], could you

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-05 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086553#comment-14086553 ] Zhijie Shen commented on YARN-1954: --- Sounds good to me Add waitFor to AMRMClient(Async)

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-04 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14084981#comment-14084981 ] Zhijie Shen commented on YARN-1954: --- My concern is that if users want a even larger

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-08-01 Thread Tsuyoshi OZAWA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14083011#comment-14083011 ] Tsuyoshi OZAWA commented on YARN-1954: -- Thanks you for the review, [~zjshen]. I'm

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-07-21 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14068411#comment-14068411 ] Zhijie Shen commented on YARN-1954: --- [~ozawa], I'm generally fine with the patch. Just

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-07-14 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14060429#comment-14060429 ] Hadoop QA commented on YARN-1954: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-07-14 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14060522#comment-14060522 ] Hadoop QA commented on YARN-1954: - {color:red}-1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-07-14 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14061624#comment-14061624 ] Zhijie Shen commented on YARN-1954: --- [~ozawa], thanks for the patch. I'll try to come

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-21 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13975687#comment-13975687 ] Hadoop QA commented on YARN-1954: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-20 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13975329#comment-13975329 ] Zhijie Shen commented on YARN-1954: --- [~ozawa], thanks for taking care of this issue. I'd

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-18 Thread Tsuyoshi OZAWA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13973839#comment-13973839 ] Tsuyoshi OZAWA commented on YARN-1954: -- [~zjshen], I added waitFor() method which

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-18 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13973850#comment-13973850 ] Hadoop QA commented on YARN-1954: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-18 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13973857#comment-13973857 ] Hadoop QA commented on YARN-1954: - {color:green}+1 overall{color}. Here are the results of

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-17 Thread Hong Zhiguo (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13973749#comment-13973749 ] Hong Zhiguo commented on YARN-1954: --- Do you mean group the status pooling loop into the

[jira] [Commented] (YARN-1954) Add waitFor to AMRMClient(Async)

2014-04-17 Thread Zhijie Shen (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13973791#comment-13973791 ] Zhijie Shen commented on YARN-1954: --- Let me give an example: On main non-daemon thread: