You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-commits@hadoop.apache.org by tu...@apache.org on 2013/05/15 00:14:15 UTC

svn commit: r1482624 - in /hadoop/common/branches/branch-2/hadoop-mapreduce-project: CHANGES.txt hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java

Author: tucu
Date: Tue May 14 22:14:15 2013
New Revision: 1482624

URL: http://svn.apache.org/r1482624
Log:
MAPREDUCE-5244. Two functions changed their visibility in JobStatus. (zjshen via tucu)

Modified:
    hadoop/common/branches/branch-2/hadoop-mapreduce-project/CHANGES.txt
    hadoop/common/branches/branch-2/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java

Modified: hadoop/common/branches/branch-2/hadoop-mapreduce-project/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-2/hadoop-mapreduce-project/CHANGES.txt?rev=1482624&r1=1482623&r2=1482624&view=diff
==============================================================================
--- hadoop/common/branches/branch-2/hadoop-mapreduce-project/CHANGES.txt (original)
+++ hadoop/common/branches/branch-2/hadoop-mapreduce-project/CHANGES.txt Tue May 14 22:14:15 2013
@@ -254,6 +254,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

Modified: hadoop/common/branches/branch-2/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-2/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java?rev=1482624&r1=1482623&r2=1482624&view=diff
==============================================================================
--- hadoop/common/branches/branch-2/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java (original)
+++ hadoop/common/branches/branch-2/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobStatus.java Tue May 14 22:14:15 2013
@@ -381,8 +381,14 @@ public class JobStatus extends org.apach
 
    /**
     * 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.apach
 
    /**
     * 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);
    }