You are viewing a plain text version of this content. The canonical link for it is here.

Posted to mapreduce-commits@hadoop.apache.org by vi...@apache.org on 2011/08/03 13:49:17 UTC

svn commit: r1153449 - /hadoop/common/branches/MR-279/mapreduce/yarn/yarn-server/yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/SchedulerApp.java

Author: vinodkv
Date: Wed Aug  3 11:49:16 2011
New Revision: 1153449

URL: http://svn.apache.org/viewvc?rev=1153449&view=rev
Log:
Fixing a bug related to releasing containers with FifoScheduler.

Modified:
    hadoop/common/branches/MR-279/mapreduce/yarn/yarn-server/yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/SchedulerApp.java

Modified: hadoop/common/branches/MR-279/mapreduce/yarn/yarn-server/yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/SchedulerApp.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/MR-279/mapreduce/yarn/yarn-server/yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/SchedulerApp.java?rev=1153449&r1=1153448&r2=1153449&view=diff
==============================================================================
--- hadoop/common/branches/MR-279/mapreduce/yarn/yarn-server/yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/SchedulerApp.java (original)
+++ hadoop/common/branches/MR-279/mapreduce/yarn/yarn-server/yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/SchedulerApp.java Wed Aug  3 11:49:16 2011
@@ -141,7 +141,7 @@ public class SchedulerApp {
         " in state: " + container.getState());
     
     // Remove from the list of containers
-    liveContainers.remove(container.getContainer());
+    liveContainers.remove(container.getContainerId());
     
     // Update usage metrics 
     Resource containerResource = container.getContainer().getResource();