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:36:17 UTC

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

Author: omalley
Date: Fri Mar  4 03:36:16 2011
New Revision: 1077056

URL: http://svn.apache.org/viewvc?rev=1077056&view=rev
Log:
commit f622ab0a03e68a9d4645fd74d0be91c5e041b7df
Author: Arun C Murthy <ac...@apache.org>
Date:   Thu Nov 19 11:52:38 2009 -0800

    MAPREDUCE-1219. Fixed JobTracker to not collect per-job metrics, thus easing load on it. Amareshwari Sriramadasu.
    
    From https://issues.apache.org/jira/secure/attachment/12425302/patch-1219-ydist.txt
    
    +++ b/YAHOO-CHANGES.txt
    +    MAPREDUCE-1219. Fixed JobTracker to not collect per-job metrics, thus
    +    easing load on it. (Amareshwari Sriramadasu via acmurthy)
    +

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

Modified: hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTrackerMetricsInst.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTrackerMetricsInst.java?rev=1077056&r1=1077055&r2=1077056&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTrackerMetricsInst.java (original)
+++ hadoop/common/branches/branch-0.20-security-patches/src/mapred/org/apache/hadoop/mapred/JobTrackerMetricsInst.java Fri Mar  4 03:36:16 2011
@@ -154,12 +154,6 @@ class JobTrackerMetricsInst extends JobT
       numTrackersBlackListed = 0;
     }
     metricsRecord.update();
-
-    if (tracker != null) {
-      for (JobInProgress jip : tracker.getRunningJobs()) {
-        jip.updateMetrics();
-      }
-    }
   }
 
   @Override