[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-25 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13881843#comment-13881843
 ] 

Hudson commented on YARN-1607:
--

FAILURE: Integrated in Hadoop-Yarn-trunk #461 (See 
[https://builds.apache.org/job/Hadoop-Yarn-trunk/461/])
YARN-1607. TestRM relies on the scheduler assigning multiple containers in a 
single node update (Sandy Ryza) (sandy: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1560533)
* /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
* 
/hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/TestRM.java


 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Fix For: 2.4.0

 Attachments: YARN-1607-1.patch, YARN-1607-2.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-25 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13881870#comment-13881870
 ] 

Hudson commented on YARN-1607:
--

FAILURE: Integrated in Hadoop-Mapreduce-trunk #1678 (See 
[https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1678/])
YARN-1607. TestRM relies on the scheduler assigning multiple containers in a 
single node update (Sandy Ryza) (sandy: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1560533)
* /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
* 
/hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/TestRM.java


 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Fix For: 2.4.0

 Attachments: YARN-1607-1.patch, YARN-1607-2.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-25 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13881888#comment-13881888
 ] 

Hudson commented on YARN-1607:
--

SUCCESS: Integrated in Hadoop-Hdfs-trunk #1653 (See 
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/1653/])
YARN-1607. TestRM relies on the scheduler assigning multiple containers in a 
single node update (Sandy Ryza) (sandy: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1560533)
* /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
* 
/hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/TestRM.java


 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Fix For: 2.4.0

 Attachments: YARN-1607-1.patch, YARN-1607-2.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-22 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13879167#comment-13879167
 ] 

Karthik Kambatla commented on YARN-1607:


Thanks Sandy. LGTM. +1, pending Jenkins.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607-1.patch, YARN-1607-2.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-22 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13879203#comment-13879203
 ] 

Hadoop QA commented on YARN-1607:
-

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12624425/YARN-1607-2.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 1 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager.

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-YARN-Build/2928//testReport/
Console output: https://builds.apache.org/job/PreCommit-YARN-Build/2928//console

This message is automatically generated.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607-1.patch, YARN-1607-2.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-22 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13879302#comment-13879302
 ] 

Hudson commented on YARN-1607:
--

SUCCESS: Integrated in Hadoop-trunk-Commit #5035 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/5035/])
YARN-1607. TestRM relies on the scheduler assigning multiple containers in a 
single node update (Sandy Ryza) (sandy: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1560533)
* /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
* 
/hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/TestRM.java


 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Fix For: 2.4.0

 Attachments: YARN-1607-1.patch, YARN-1607-2.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-21 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877320#comment-13877320
 ] 

Hadoop QA commented on YARN-1607:
-

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12624090/YARN-1607.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 1 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager.

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-YARN-Build/2919//testReport/
Console output: https://builds.apache.org/job/PreCommit-YARN-Build/2919//console

This message is automatically generated.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-21 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877643#comment-13877643
 ] 

Karthik Kambatla commented on YARN-1607:


Not introduced by this patch, but the while() loops without a timeout (waiting 
for the right number of containers or otherwise) is an easy way to wait forever 
especially in a test case. Can we have bounds on how long we can be in each of 
those loops? Further, would it make sense to lower the sleep intervals further? 

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-21 Thread Sandy Ryza (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877766#comment-13877766
 ] 

Sandy Ryza commented on YARN-1607:
--

For the timeouts, I think the cleanest would be to add timeouts to the tests.  
I think some of the sleep intervals could be lowered further.  Will attach a 
patch that does these.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-21 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877801#comment-13877801
 ] 

Karthik Kambatla commented on YARN-1607:


Looks mostly good. One more nit: instead of using inconsistent literal values 
for sleep intervals, can we define a constant SLEEP_INTERVAL and use it through 
out. Thanks.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607-1.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-21 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877803#comment-13877803
 ] 

Karthik Kambatla commented on YARN-1607:


I don't know the reason why, but I was told (forget by whom) that using 
timeouts is discouraged. Might want to check that too.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607-1.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-21 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877804#comment-13877804
 ] 

Karthik Kambatla commented on YARN-1607:


I do agree timeouts are the easiest way to go here, and the timeouts are high 
enough in the patch that it shouldn't cause problems on slow machines. 

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607-1.patch, YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1607) TestRM expects the capacity scheduler

2014-01-20 Thread Sandy Ryza (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877304#comment-13877304
 ] 

Sandy Ryza commented on YARN-1607:
--

The attached patch makes the tests agnostic to the scheduler being used by not 
expecting multiple containers to come on a single heartbeat.

 TestRM expects the capacity scheduler
 -

 Key: YARN-1607
 URL: https://issues.apache.org/jira/browse/YARN-1607
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sandy Ryza
Assignee: Sandy Ryza
 Attachments: YARN-1607.patch


 We should either explicitly set the Capacity Scheduler or make it 
 scheduler-agnostic



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)