From 08479378c2a777759d9b9ea1884aaba92eb18948 Mon Sep 17 00:00:00 2001 From: Vinod Kumar Vavilapalli Date: Mon, 6 Feb 2012 23:16:37 +0000 Subject: [PATCH] MAPREDUCE-3818. Fixed broken compilation in TestSubmitJob after the patch for HDFS-2895. Contributed by Suresh Srinivas. git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1241253 13f79535-47bb-0310-9956-ffa450edef68 --- hadoop-mapreduce-project/CHANGES.txt | 3 +++ .../apache/hadoop/mapred/TestSubmitJob.java | 18 ++++++------------ 2 files changed, 9 insertions(+), 12 deletions(-) diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt index f74b1f96edc..fd49b7b0cc9 100644 --- a/hadoop-mapreduce-project/CHANGES.txt +++ b/hadoop-mapreduce-project/CHANGES.txt @@ -87,6 +87,9 @@ Trunk (unreleased changes) MAPREDUCE-3740. Fixed broken mapreduce compilation after the patch for HADOOP-7965. (Devaraj K via vinodkv) + MAPREDUCE-3818. Fixed broken compilation in TestSubmitJob after the patch + for HDFS-2895. (Suresh Srinivas via vinodkv) + Release 0.23.1 - Unreleased INCOMPATIBLE CHANGES diff --git a/hadoop-mapreduce-project/src/test/mapred/org/apache/hadoop/mapred/TestSubmitJob.java b/hadoop-mapreduce-project/src/test/mapred/org/apache/hadoop/mapred/TestSubmitJob.java index c32ead05b8a..488ab7664b6 100644 --- a/hadoop-mapreduce-project/src/test/mapred/org/apache/hadoop/mapred/TestSubmitJob.java +++ b/hadoop-mapreduce-project/src/test/mapred/org/apache/hadoop/mapred/TestSubmitJob.java @@ -33,7 +33,7 @@ import org.apache.hadoop.fs.Path; import org.apache.hadoop.hdfs.MiniDFSCluster; import org.apache.hadoop.hdfs.protocol.HdfsFileStatus; -import org.apache.hadoop.hdfs.protocolR23Compatible.ClientNamenodeWireProtocol; +import org.apache.hadoop.hdfs.protocolPB.ClientNamenodeProtocolTranslatorPB; import org.apache.hadoop.hdfs.server.namenode.NameNode; import org.apache.hadoop.ipc.RPC; import org.apache.hadoop.ipc.RemoteException; @@ -148,16 +148,10 @@ static ClientProtocol getJobSubmitClient(JobConf conf, conf, NetUtils.getSocketFactory(conf, ClientProtocol.class)); } - static ClientNamenodeWireProtocol getDFSClient( - Configuration conf, UserGroupInformation ugi) - throws IOException { - return (ClientNamenodeWireProtocol) - RPC.getProxy(ClientNamenodeWireProtocol.class, - ClientNamenodeWireProtocol.versionID, - NameNode.getAddress(conf), ugi, - conf, - NetUtils.getSocketFactory(conf, - ClientNamenodeWireProtocol.class)); + static org.apache.hadoop.hdfs.protocol.ClientProtocol getDFSClient( + Configuration conf, UserGroupInformation ugi) throws IOException { + return new ClientNamenodeProtocolTranslatorPB(NameNode.getAddress(conf), + conf, ugi); } /** @@ -226,7 +220,7 @@ public RunningJob run() throws IOException { UserGroupInformation user2 = TestMiniMRWithDFSWithDistinctUsers.createUGI("user2", false); JobConf conf_other = mr.createJobConf(); - ClientNamenodeWireProtocol client = + org.apache.hadoop.hdfs.protocol.ClientProtocol client = getDFSClient(conf_other, user2); // try accessing mapred.system.dir/jobid/*