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 sh...@apache.org on 2009/07/31 10:38:26 UTC

svn commit: r799545 - in /hadoop/mapreduce/trunk: CHANGES.txt src/test/mapred/org/apache/hadoop/mapred/TestNodeRefresh.java

Author: sharad
Date: Fri Jul 31 08:38:26 2009
New Revision: 799545

URL: http://svn.apache.org/viewvc?rev=799545&view=rev
Log:
MAPREDUCE-760. Fix a timing issue in TestNodeRefresh. Contributed by Amar Kamat.

Modified:
    hadoop/mapreduce/trunk/CHANGES.txt
    hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestNodeRefresh.java

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=799545&r1=799544&r2=799545&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Fri Jul 31 08:38:26 2009
@@ -295,3 +295,7 @@
 
     MAPREDUCE-792. Fix unchecked warnings in DBInputFormat.  (Aaron Kimball
     via szetszwo)
+
+    MAPREDUCE-760. Fix a timing issue in TestNodeRefresh. (Amar Kamat via 
+    sharad)
+

Modified: hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestNodeRefresh.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestNodeRefresh.java?rev=799545&r1=799544&r2=799545&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestNodeRefresh.java (original)
+++ hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestNodeRefresh.java Fri Jul 31 08:38:26 2009
@@ -428,10 +428,6 @@
     mr.stopJobTracker();
     mr.startJobTracker();
     
-    // start a tracker so that the jobs move to completion
-    String newTrackerHostName = getHostname(2);
-    mr.startTaskTracker(newTrackerHostName, null, 2, 1);
-
     // Wait for the JT to be ready
     UtilsForTests.waitForJobTracker(jobClient);
 
@@ -444,7 +440,7 @@
     
     // check the cluster status and tracker size
     assertEquals("Tracker is not lost upon host decommissioning", 
-                 1, jt.getClusterStatus(false).getTaskTrackers());
+                 0, jt.getClusterStatus(false).getTaskTrackers());
     assertEquals("Excluded node count is incorrect", 
                  1, jt.getClusterStatus(false).getNumExcludedNodes());
     
@@ -454,6 +450,10 @@
                   status.getHost().equals(hostToDecommission));
     }
 
+    // start a tracker so that the jobs move to completion
+    String newTrackerHostName = getHostname(2);
+    mr.startTaskTracker(newTrackerHostName, null, 2, 1);
+    
     // wait for the job
     job.waitForCompletion();