Author: bobby Date: Thu Jan 3 17:09:18 2013 New Revision: 1428482 URL: http://svn.apache.org/viewvc?rev=1428482&view=rev Log: MAPREDUCE-4279. getClusterStatus() fails with null pointer exception when running jobs in local mode (Devaraj K via bobby)
Added: hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/test/java/org/apache/hadoop/mapred/TestJobClient.java Modified: hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapred/LocalJobRunner.java Modified: hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt?rev=1428482&r1=1428481&r2=1428482&view=diff ============================================================================== --- hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt (original) +++ hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt Thu Jan 3 17:09:18 2013 @@ -656,6 +656,9 @@ Release 0.23.6 - UNRELEASED MAPREDUCE-4813. AM timing out during job commit (jlowe via bobby) + MAPREDUCE-4279. getClusterStatus() fails with null pointer exception when + running jobs in local mode (Devaraj K via bobby) + Release 0.23.5 - UNRELEASED INCOMPATIBLE CHANGES Modified: hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapred/LocalJobRunner.java URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapred/LocalJobRunner.java?rev=1428482&r1=1428481&r2=1428482&view=diff ============================================================================== --- hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapred/LocalJobRunner.java (original) +++ hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapred/LocalJobRunner.java Thu Jan 3 17:09:18 2013 @@ -67,7 +67,6 @@ import com.google.common.util.concurrent /** Implements MapReduce locally, in-process, for debugging. */ @InterfaceAudience.Private @InterfaceStability.Unstable -@SuppressWarnings("deprecation") public class LocalJobRunner implements ClientProtocol { public static final Log LOG = LogFactory.getLog(LocalJobRunner.class); @@ -686,7 +685,7 @@ public class LocalJobRunner implements C */ public TaskTrackerInfo[] getActiveTrackers() throws IOException, InterruptedException { - return null; + return new TaskTrackerInfo[0]; } /** @@ -695,7 +694,7 @@ public class LocalJobRunner implements C */ public TaskTrackerInfo[] getBlacklistedTrackers() throws IOException, InterruptedException { - return null; + return new TaskTrackerInfo[0]; } public TaskCompletionEvent[] getTaskCompletionEvents( Added: hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/test/java/org/apache/hadoop/mapred/TestJobClient.java URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/test/java/org/apache/hadoop/mapred/TestJobClient.java?rev=1428482&view=auto ============================================================================== --- hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/test/java/org/apache/hadoop/mapred/TestJobClient.java (added) +++ hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/test/java/org/apache/hadoop/mapred/TestJobClient.java Thu Jan 3 17:09:18 2013 @@ -0,0 +1,46 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.hadoop.mapred; + +import java.util.Collection; + +import org.apache.hadoop.conf.Configuration; +import org.apache.hadoop.mapred.ClusterStatus.BlackListInfo; +import org.apache.hadoop.mapreduce.MRConfig; +import org.apache.hadoop.mapreduce.server.jobtracker.JTConfig; +import org.junit.Assert; +import org.junit.Test; + +public class TestJobClient { + @Test + public void testGetClusterStatusWithLocalJobRunner() throws Exception { + Configuration conf = new Configuration(); + conf.set(JTConfig.JT_IPC_ADDRESS, MRConfig.LOCAL_FRAMEWORK_NAME); + conf.set(MRConfig.FRAMEWORK_NAME, MRConfig.LOCAL_FRAMEWORK_NAME); + JobClient client = new JobClient(conf); + ClusterStatus clusterStatus = client.getClusterStatus(true); + Collection<String> activeTrackerNames = clusterStatus + .getActiveTrackerNames(); + Assert.assertEquals(0, activeTrackerNames.size()); + int blacklistedTrackers = clusterStatus.getBlacklistedTrackers(); + Assert.assertEquals(0, blacklistedTrackers); + Collection<BlackListInfo> blackListedTrackersInfo = clusterStatus + .getBlackListedTrackersInfo(); + Assert.assertEquals(0, blackListedTrackersInfo.size()); + } +}