hadoop-mapreduce-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From yhema...@apache.org
Subject svn commit: r792490 - in /hadoop/mapreduce/trunk: CHANGES.txt src/java/org/apache/hadoop/mapred/JobTracker.java
Date Thu, 09 Jul 2009 10:41:12 GMT
Author: yhemanth
Date: Thu Jul  9 10:41:12 2009
New Revision: 792490

URL: http://svn.apache.org/viewvc?rev=792490&view=rev
Log:
MAPREDUCE-732. Removed spurious log statements in the node blacklisting logic. Contributed
by Sreekanth Ramakrishnan.

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

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=792490&r1=792489&r2=792490&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Thu Jul  9 10:41:12 2009
@@ -155,3 +155,7 @@
 
     HADOOP-6090. Updates gridmix script to use new mapreduce api output 
     format. (Amareshwari Sriramadasu via sharad)
+
+    MAPREDUCE-732. Removed spurious log statements in the node
+    blacklisting logic. (Sreekanth Ramakrishnan via yhemanth)
+

Modified: hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java?rev=792490&r1=792489&r2=792490&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java (original)
+++ hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java Thu Jul  9 10:41:12
2009
@@ -609,8 +609,14 @@
       FaultInfo fi = getFaultInfo(hostName, true);
       boolean blackListed = fi.isBlacklisted();
       if(blackListed) {
-        LOG.info("Adding blacklisted reason for tracker : " + hostName 
-            + " Reason for blacklisting is : " + rfb);
+        if (LOG.isDebugEnabled()) {
+          LOG.debug("Adding blacklisted reason for tracker : " + hostName 
+              + " Reason for blacklisting is : " + rfb);
+        }
+        if (!fi.getReasonforblacklisting().contains(rfb)) {
+          LOG.info("Adding blacklisted reason for tracker : " + hostName
+              + " Reason for blacklisting is : " + rfb);
+        }
         fi.addBlackListedReason(rfb, reason);
       } else {
         LOG.info("Blacklisting tracker : " + hostName 



Mime
View raw message