Author: cutting
Date: Tue Jan 23 13:36:40 2007
New Revision: 499140

URL: http://svn.apache.org/viewvc?view=rev&rev=499140
Log:
HADOOP-912.  Fix a bug in TaskTracker.isIdle() that was sporadically causing 
unit test failures.  Contributed by Arun.

Modified:
    lucene/hadoop/trunk/CHANGES.txt
    lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/TaskTracker.java

Modified: lucene/hadoop/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?view=diff&rev=499140&r1=499139&r2=499140
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Tue Jan 23 13:36:40 2007
@@ -57,6 +57,9 @@
 17. HADOOP-901.  Add support for recursive renaming to the S3 filesystem.
     (Tom White via cutting)
 
+18. HADOOP-912.  Fix a bug in TaskTracker.isIdle() that was
+    sporadically causing unit test failures.  (Arun C Murthy via cutting)
+
 
 Release 0.10.1 - 2007-01-10
 

Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/TaskTracker.java
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/TaskTracker.java?view=diff&rev=499140&r1=499139&r2=499140
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/TaskTracker.java 
(original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/TaskTracker.java Tue 
Jan 23 13:36:40 2007
@@ -1475,7 +1475,7 @@
      * @return has this task tracker finished and cleaned up all of its tasks?
      */
     public synchronized boolean isIdle() {
-      return tasks.isEmpty();
+      return tasks.isEmpty() && tasksToCleanup.isEmpty();
     }
     
     /**


Reply via email to