From 7db82337225b346f73d072d971363d6823010de7 Mon Sep 17 00:00:00 2001
From: Alejandro Abdelnur <tucu@apache.org>
Date: Mon, 13 May 2013 23:09:05 +0000
Subject: [PATCH] MAPREDUCE-5220. Setter methods in TaskCompletionEvent are
 public in MR1 and protected in MR2. (sandyr via tucu)

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1482143 13f79535-47bb-0310-9956-ffa450edef68
---
 hadoop-mapreduce-project/CHANGES.txt                |  3 +++
 .../apache/hadoop/mapred/TaskCompletionEvent.java   | 13 +++++++++----
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt
index d0079590504..b61f81aae23 100644
--- a/hadoop-mapreduce-project/CHANGES.txt
+++ b/hadoop-mapreduce-project/CHANGES.txt
@@ -245,6 +245,9 @@ Release 2.0.5-beta - UNRELEASED
     MAPREDUCE-5208. Modified ShuffleHandler to use SecureIOUtils for reading
     local files. (Omkar Vinit Joshi via vinodkv)
 
+    MAPREDUCE-5220. Setter methods in TaskCompletionEvent are public in MR1 and 
+    protected in MR2. (sandyr via tucu)
+
 Release 2.0.4-alpha - 2013-04-25
 
   INCOMPATIBLE CHANGES
diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/TaskCompletionEvent.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/TaskCompletionEvent.java
index 9f25339a646..02cba94c4ee 100644
--- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/TaskCompletionEvent.java
+++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/TaskCompletionEvent.java
@@ -19,6 +19,7 @@
 package org.apache.hadoop.mapred;
 
 import org.apache.hadoop.classification.InterfaceAudience;
+import org.apache.hadoop.classification.InterfaceAudience.Private;
 import org.apache.hadoop.classification.InterfaceStability;
 
 /**
@@ -117,7 +118,8 @@ public class TaskCompletionEvent
    * Set task status. 
    * @param status
    */
-  protected void setTaskStatus(Status status) {
+  @Private
+  public void setTaskStatus(Status status) {
     super.setTaskStatus(org.apache.hadoop.mapreduce.
       TaskCompletionEvent.Status.valueOf(status.name()));
   }
@@ -126,7 +128,8 @@ public class TaskCompletionEvent
    * Set the task completion time
    * @param taskCompletionTime time (in millisec) the task took to complete
    */
-  protected void setTaskRunTime(int taskCompletionTime) {
+  @Private
+  public void setTaskRunTime(int taskCompletionTime) {
     super.setTaskRunTime(taskCompletionTime);
   }
 
@@ -134,7 +137,8 @@ public class TaskCompletionEvent
    * set event Id. should be assigned incrementally starting from 0. 
    * @param eventId
    */
-  protected void setEventId(int eventId) {
+  @Private
+  public void setEventId(int eventId) {
     super.setEventId(eventId);
   }
 
@@ -142,7 +146,8 @@ public class TaskCompletionEvent
    * Set task tracker http location. 
    * @param taskTrackerHttp
    */
-  protected void setTaskTrackerHttp(String taskTrackerHttp) {
+  @Private
+  public void setTaskTrackerHttp(String taskTrackerHttp) {
     super.setTaskTrackerHttp(taskTrackerHttp);
   }
 }