HADOOP-4380. Made several new classes (Child, JVMId,

JobTrackerInstrumentation, QueueManager, ResourceEstimator, 
TaskTrackerInstrumentation, and TaskTrackerMetricsInst) in 
org.apache.hadoop.mapred  package private instead of public. (omalley)


git-svn-id: https://svn.apache.org/repos/asf/hadoop/core/branches/branch-0.19@703509 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -471,6 +471,11 @@ Release 0.19.0 - Unreleased
     HADOOP-4225. FSEditLog.logOpenFile() should persist accessTime 
     rather than modificationTime. (shv)
 
+    HADOOP-4380. Made several new classes (Child, JVMId, 
+    JobTrackerInstrumentation, QueueManager, ResourceEstimator, 
+    TaskTrackerInstrumentation, and TaskTrackerMetricsInst) in 
+    org.apache.hadoop.mapred  package private instead of public. (omalley)
+
   BUG FIXES
 
     HADOOP-3563.  Refactor the distributed upgrade code so that it is 

src/mapred/org/apache/hadoop/mapred/Child.java

@@ -41,7 +41,7 @@ import org.apache.log4j.LogManager;
  * The main() for child processes. 
  */
 
-public class Child {
+class Child {
 
   public static final Log LOG =
     LogFactory.getLog(TaskTracker.class);

src/mapred/org/apache/hadoop/mapred/JVMId.java

@@ -23,7 +23,7 @@ import java.io.DataOutput;
 import java.io.IOException;
 import java.text.NumberFormat;
 
-public class JVMId extends ID {
+class JVMId extends ID {
   boolean isMap;
   JobID jobId;
   private static final String JVM = "jvm";

src/mapred/org/apache/hadoop/mapred/JobTrackerInstrumentation.java

@@ -17,9 +17,7 @@
  */
 package org.apache.hadoop.mapred;
 
-import org.apache.hadoop.metrics.Updater;
-
-public class JobTrackerInstrumentation {
+class JobTrackerInstrumentation {
 
   protected final JobTracker tracker;
   

src/mapred/org/apache/hadoop/mapred/QueueManager.java

@@ -19,7 +19,6 @@
 package org.apache.hadoop.mapred;
 
 import java.util.ArrayList;
-import java.util.Collection;
 import java.util.HashMap;
 import java.util.Set;
 import java.util.TreeSet;
@@ -45,7 +44,7 @@ import org.apache.hadoop.security.UserGroupInformation;
  * queues. Where such a case exists, the queue name must be used to link 
  * the common properties with the scheduler specific ones.  
  */
-public class QueueManager {
+class QueueManager {
   
   private static final Log LOG = LogFactory.getLog(QueueManager.class);
   

src/mapred/org/apache/hadoop/mapred/ResourceEstimator.java

@@ -17,7 +17,6 @@
  */
 package org.apache.hadoop.mapred;
 
-import java.util.concurrent.atomic.*;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
@@ -29,7 +28,7 @@ import org.apache.commons.logging.LogFactory;
  * There is one ResourceEstimator per JobInProgress
  *
  */
-public class ResourceEstimator {
+class ResourceEstimator {
 
   //Log with JobInProgress
   private static final Log LOG = LogFactory.getLog(

src/mapred/org/apache/hadoop/mapred/TaskTrackerInstrumentation.java

@@ -31,7 +31,7 @@ import java.io.File;
  * wouldn't have direct access to the associated TaskTracker.
  *  
  **/
-public class TaskTrackerInstrumentation  {
+class TaskTrackerInstrumentation  {
 
   protected final TaskTracker tt;
   

src/mapred/org/apache/hadoop/mapred/TaskTrackerMetricsInst.java

@@ -24,7 +24,8 @@ import org.apache.hadoop.metrics.MetricsUtil;
 import org.apache.hadoop.metrics.Updater;
 import org.apache.hadoop.metrics.jvm.JvmMetrics;
   
-public class TaskTrackerMetricsInst extends TaskTrackerInstrumentation implements Updater {
+class TaskTrackerMetricsInst extends TaskTrackerInstrumentation 
+                             implements Updater {
   private final MetricsRecord metricsRecord;
   private int numCompletedTasks = 0;
   private int timedoutTasks = 0;