You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by dd...@apache.org on 2009/01/08 06:43:00 UTC

svn commit: r732613 - in /hadoop/core/trunk: CHANGES.txt src/mapred/org/apache/hadoop/mapred/TaskTracker.java

Author: ddas
Date: Wed Jan  7 21:43:00 2009
New Revision: 732613

URL: http://svn.apache.org/viewvc?rev=732613&view=rev
Log:
HADOOP-4963. Fixes a logging to do with getting the location of  map output file. Contributed by Amareshwari Sriramadasu.

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

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=732613&r1=732612&r2=732613&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Wed Jan  7 21:43:00 2009
@@ -26,6 +26,9 @@
   OPTIMIZATIONS
 
   BUG FIXES
+    
+    HADOOP-4963. Fixes a logging to do with getting the location of
+    map output file. (Amareshwari Sriramadasu via ddas)
 
 Release 0.20.0 - Unreleased
 

Modified: hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/TaskTracker.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/TaskTracker.java?rev=732613&r1=732612&r2=732613&view=diff
==============================================================================
--- hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/TaskTracker.java (original)
+++ hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/TaskTracker.java Wed Jan  7 21:43:00 2009
@@ -1559,6 +1559,11 @@
       if(tip == null)
          return -1;
       
+      if (!tip.getTask().isMapTask() || 
+          tip.getRunState() != TaskStatus.State.SUCCEEDED) {
+        return -1;
+      }
+      
       MapOutputFile mapOutputFile = new MapOutputFile();
       mapOutputFile.setJobId(taskId.getJobID());
       mapOutputFile.setConf(conf);