svn commit: r540473 - in /lucene/hadoop/branches/branch-0.13: CHANGES.txt src/java/org/apache/hadoop/mapred/JobInProgress.java

[tomwhite]

Author: tomwhite
Date: Tue May 22 01:35:10 2007
New Revision: 540473

URL: http://svn.apache.org/viewvc?view=rev&rev=540473
Log:
Merge -r 540468:540469 from trunk to 0.13 branch. Fixes: HADOOP-1407.

Modified:
    lucene/hadoop/branches/branch-0.13/CHANGES.txt
    
lucene/hadoop/branches/branch-0.13/src/java/org/apache/hadoop/mapred/JobInProgress.java

Modified: lucene/hadoop/branches/branch-0.13/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/branches/branch-0.13/CHANGES.txt?view=diff&rev=540473&r1=540472&r2=540473
==============================================================================
--- lucene/hadoop/branches/branch-0.13/CHANGES.txt (original)
+++ lucene/hadoop/branches/branch-0.13/CHANGES.txt Tue May 22 01:35:10 2007
@@ -415,6 +415,10 @@
      force applications to use "." to name the default directory.
      (Hairong Kuang via cutting)
 
+125. HADOOP-1407.  Fix integer division bug in JobInProgress which
+     meant failed tasks didn't cause the job to fail.
+     (Arun C Murthy via tomwhite)
+
 
 Release 0.12.3 - 2007-04-06
 

Modified: 
lucene/hadoop/branches/branch-0.13/src/java/org/apache/hadoop/mapred/JobInProgress.java
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/branches/branch-0.13/src/java/org/apache/hadoop/mapred/JobInProgress.java?view=diff&rev=540473&r1=540472&r2=540473
==============================================================================
--- 
lucene/hadoop/branches/branch-0.13/src/java/org/apache/hadoop/mapred/JobInProgress.java
 (original)
+++ 
lucene/hadoop/branches/branch-0.13/src/java/org/apache/hadoop/mapred/JobInProgress.java
 Tue May 22 01:35:10 2007
@@ -945,8 +945,8 @@
       //
       boolean killJob = 
         tip.isMapTask() ? 
-            (((++failedMapTIPs*100)/numMapTasks) > mapFailuresPercent) :
-            (((++failedReduceTIPs*100)/numReduceTasks) > 
reduceFailuresPercent);
+            ((++failedMapTIPs*100) > (mapFailuresPercent*numMapTasks)) :
+            ((++failedReduceTIPs*100) > 
(reduceFailuresPercent*numReduceTasks));
       
       if (killJob) {
         LOG.info("Aborting job " + profile.getJobId());