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 om...@apache.org on 2011/03/04 04:30:54 UTC

svn commit: r1077020 - /hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTracker.java

Author: omalley
Date: Fri Mar  4 03:30:54 2011
New Revision: 1077020

URL: http://svn.apache.org/viewvc?rev=1077020&view=rev
Log:
commit beba981b072ff739ffd858a48e9d2fda65450024
Author: Hemanth Yamijala <yh...@apache.org>
Date:   Thu Oct 15 16:05:41 2009 +0530

    MAPREDUCE:732 from http://issues.apache.org/jira/secure/attachment/12413001/MAPRED-732-ydist.patch
    
    +++ b/YAHOO-CHANGES.txt
    +68. MAPREDUCE-732. Removed spurious log statements in the node
    +    blacklisting logic. (Sreekanth Ramakrishnan via yhemanth)
    +

Modified:
    hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTracker.java

Modified: hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTracker.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTracker.java?rev=1077020&r1=1077019&r2=1077020&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTracker.java (original)
+++ hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTracker.java Fri Mar  4 03:30:54 2011
@@ -722,8 +722,14 @@ public class JobTracker implements MRCon
       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