svn commit: r573513 - in /lucene/hadoop/trunk: CHANGES.txt src/java/org/apache/hadoop/mapred/JobTracker.java

[acmurthy]

Author: acmurthy
Date: Fri Sep  7 02:09:00 2007
New Revision: 573513

URL: http://svn.apache.org/viewvc?rev=573513&view=rev
Log:
HADOOP-1018.  Improve documentation w.r.t handling of lost hearbeats between 
TaskTrackers and JobTracker.

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

Modified: lucene/hadoop/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?rev=573513&r1=573512&r2=573513&view=diff
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Fri Sep  7 02:09:00 2007
@@ -174,6 +174,9 @@
     code in Hadoop so that you can copy from and to all of the supported file 
     systems.(Chris Douglas via omalley)
 
+    HADOOP-1018.  Improve documentation w.r.t handling of lost hearbeats 
between 
+    TaskTrackers and JobTracker. (acmurthy)
+
 Release 0.14.1 - 2007-09-04
 
   BUG FIXES

Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java?rev=573513&r1=573512&r2=573513&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java 
(original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java Fri 
Sep  7 02:09:00 2007
@@ -1101,13 +1101,14 @@
 
       }
                 
-      // It is completely safe to ignore a 'duplicate' from a tracker
-      // since we are guaranteed that the tracker sends the same 
-      // 'heartbeat' when rpcs are lost. 
-      // [EMAIL PROTECTED] TaskTracker.transmitHeartbeat()}
+      // It is completely safe to not process a 'duplicate' heartbeat from a 
+      // [EMAIL PROTECTED] TaskTracker} since it resends the heartbeat when 
rpcs are lost - 
+      // @see [EMAIL PROTECTED] TaskTracker.transmitHeartbeat()};
+      // acknowledge it by re-sending the previous response to let the 
+      // [EMAIL PROTECTED] TaskTracker} go forward. 
       if (prevHeartbeatResponse.getResponseId() != responseId) {
         LOG.info("Ignoring 'duplicate' heartbeat from '" + 
-                 trackerName + "'");
+                 trackerName + "'; resending the previous 'lost' response");
         return prevHeartbeatResponse;
       }
     }