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(); } /**