Bläddra i källkod

MAPREDUCE-5244. Two functions changed their visibility in JobStatus. (zjshen via tucu)

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1482622 13f79535-47bb-0310-9956-ffa450edef68
Alejandro Abdelnur 12 år sedan
förälder
incheckning
3472800eb6

+ 3 - 0
hadoop-mapreduce-project/CHANGES.txt

@@ -416,6 +416,9 @@ Release 2.0.5-beta - UNRELEASED
     MAPREDUCE-5240. Fix a bug in MRAppMaster because of which OutputCommitter
     could not access credentials set by the user. (vinodkv)
 
+    MAPREDUCE-5244. Two functions changed their visibility in JobStatus. 
+    (zjshen via tucu)
+
 Release 2.0.4-alpha - 2013-04-25
 
   INCOMPATIBLE CHANGES

+ 13 - 3
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java

@@ -381,8 +381,14 @@ public class JobStatus extends org.apache.hadoop.mapreduce.JobStatus {
 
    /**
     * Change the current run state of the job.
+    *
+    * The setter is public to be compatible with M/R 1.x, however, it should be
+    * used internally.
+    *
+    * @param state the state of the job
     */
-   protected synchronized void setRunState(int state) {
+   @InterfaceAudience.Private
+   public synchronized void setRunState(int state) {
      super.setState(getEnum(state));
    }
 
@@ -409,10 +415,14 @@ public class JobStatus extends org.apache.hadoop.mapreduce.JobStatus {
 
    /**
     * Used to set the scheduling information associated to a particular Job.
-    * 
+    *
+    * The setter is public to be compatible with M/R 1.x, however, it should be
+    * used internally.
+    *
     * @param schedulingInfo Scheduling information of the job
     */
-   protected synchronized void setSchedulingInfo(String schedulingInfo) {
+   @InterfaceAudience.Private
+   public synchronized void setSchedulingInfo(String schedulingInfo) {
      super.setSchedulingInfo(schedulingInfo);
    }