Author: sradia Date: Wed Jul 4 17:56:51 2012 New Revision: 1357351 URL: http://svn.apache.org/viewvc?rev=1357351&view=rev Log: HADOOP-8487 Many HDFS tests use a test path intended for local file system tests (Ivan Mitic via Sanjay Radia)
Modified: hadoop/common/branches/branch-1-win/CHANGES.txt hadoop/common/branches/branch-1-win/src/contrib/capacity-scheduler/src/test/org/apache/hadoop/mapred/ClusterWithCapacityScheduler.java hadoop/common/branches/branch-1-win/src/contrib/test/mapred-site.xml hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/FileSystemTestHelper.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/TestFSMainOperationsLocalFileSystem.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/TestBBWBlockReport.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobInProgress.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobQueueInformation.java hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestSubmitJob.java Modified: hadoop/common/branches/branch-1-win/CHANGES.txt URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/CHANGES.txt?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/CHANGES.txt (original) +++ hadoop/common/branches/branch-1-win/CHANGES.txt Wed Jul 4 17:56:51 2012 @@ -48,6 +48,8 @@ branch-hadoop-1-win - unreleased HADOOP-8534 Some tests leave a config file open causing failure on windows (Ivan Mitic via Sanjay Radia) + HADOOP-8487 Many HDFS tests use a test path intended for local file system tests (Ivan Mitic via Sanjay Radia) + Release 1.1.0 - unreleased NEW FEATURES Modified: hadoop/common/branches/branch-1-win/src/contrib/capacity-scheduler/src/test/org/apache/hadoop/mapred/ClusterWithCapacityScheduler.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/contrib/capacity-scheduler/src/test/org/apache/hadoop/mapred/ClusterWithCapacityScheduler.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/contrib/capacity-scheduler/src/test/org/apache/hadoop/mapred/ClusterWithCapacityScheduler.java (original) +++ hadoop/common/branches/branch-1-win/src/contrib/capacity-scheduler/src/test/org/apache/hadoop/mapred/ClusterWithCapacityScheduler.java Wed Jul 4 17:56:51 2012 @@ -222,7 +222,7 @@ public class ClusterWithCapacitySchedule // Check the resourcePath directory File file = new File(resourcePath, name); if (file.exists()) { - return new URL("file://" + file.getAbsolutePath()); + return new URL("file", null, file.getAbsolutePath()); } } catch (MalformedURLException mue) { LOG.warn("exception : " + mue); Modified: hadoop/common/branches/branch-1-win/src/contrib/test/mapred-site.xml URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/contrib/test/mapred-site.xml?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/contrib/test/mapred-site.xml (original) +++ hadoop/common/branches/branch-1-win/src/contrib/test/mapred-site.xml Wed Jul 4 17:56:51 2012 @@ -10,4 +10,9 @@ <value>build/contrib/${contrib.name}/test/system</value> </property> +<property> + <name>mapreduce.jobtracker.staging.root.dir</name> + <value>build/contrib/${contrib.name}/test/staging</value> +</property> + </configuration> Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/FileSystemTestHelper.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/FileSystemTestHelper.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/FileSystemTestHelper.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/FileSystemTestHelper.java Wed Jul 4 17:56:51 2012 @@ -31,7 +31,7 @@ import org.junit.Assert; */ public final class FileSystemTestHelper { // The test root is relative to the <wd>/build/test/data by default - public static final String TEST_ROOT_DIR = + public static String TEST_ROOT_DIR = System.getProperty("test.build.data", "build/test/data") + "/test"; private static final int DEFAULT_BLOCK_SIZE = 1024; private static final int DEFAULT_NUM_BLOCKS = 2; @@ -67,7 +67,7 @@ public final class FileSystemTestHelper public static String getAbsoluteTestRootDir(FileSystem fSys) throws IOException { if (absTestRootDir == null) { - if (TEST_ROOT_DIR.startsWith("/")) { + if (new Path(TEST_ROOT_DIR).isAbsolute()) { absTestRootDir = TEST_ROOT_DIR; } else { absTestRootDir = fSys.getWorkingDirectory().toString() + "/" Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/TestFSMainOperationsLocalFileSystem.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/TestFSMainOperationsLocalFileSystem.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/TestFSMainOperationsLocalFileSystem.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/fs/TestFSMainOperationsLocalFileSystem.java Wed Jul 4 17:56:51 2012 @@ -21,6 +21,7 @@ package org.apache.hadoop.fs; import java.io.IOException; import org.apache.hadoop.conf.Configuration; +import org.junit.After; import org.junit.Before; public class TestFSMainOperationsLocalFileSystem extends @@ -32,6 +33,11 @@ public class TestFSMainOperationsLocalFi super.setUp(); } + @After + public void tearDown() throws Exception { + super.tearDown(); + } + static Path wd = null; protected Path getDefaultWorkingDirectory() throws IOException { if (wd == null) Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/TestBBWBlockReport.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/TestBBWBlockReport.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/TestBBWBlockReport.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/TestBBWBlockReport.java Wed Jul 4 17:56:51 2012 @@ -33,9 +33,8 @@ import org.junit.Before; import org.junit.Test; public class TestBBWBlockReport { - - private final Path src = new Path(System.getProperty("test.build.data", - "/tmp"), "testfile"); + private final Path src = + new Path("/tmp/TestBBWBlockReport", "testfile"); private Configuration conf = null; Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java Wed Jul 4 17:56:51 2012 @@ -44,8 +44,8 @@ import static org.apache.hadoop.test.Met public class TestNameNodeMetrics extends TestCase { private static final Configuration CONF = new Configuration(); private static final int DFS_REPLICATION_INTERVAL = 1; - private static final Path TEST_ROOT_DIR_PATH = - new Path(System.getProperty("test.build.data", "build/test/data")); + private static final Path TEST_ROOT_DIR_PATH = + new Path("/tmp/TestNameNodeMetrics"); // Number of datanodes in the cluster private static final int DATANODE_COUNT = 3; Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java Wed Jul 4 17:56:51 2012 @@ -29,6 +29,7 @@ import org.apache.commons.logging.impl.L import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.FSMainOperationsBaseTest; import org.apache.hadoop.fs.FileSystem; +import org.apache.hadoop.fs.FileSystemTestHelper; import org.apache.hadoop.fs.Path; import org.apache.hadoop.fs.permission.FsPermission; import org.apache.hadoop.hdfs.DFSConfigKeys; @@ -54,6 +55,10 @@ public class TestFSMainOperationsWebHdfs @BeforeClass public static void setupCluster() { + // Initialize the test root directory to a DFS like path + // since we are testing based on the MiniDFSCluster. + FileSystemTestHelper.TEST_ROOT_DIR = "/tmp/TestFSMainOperationsWebHdfs"; + final Configuration conf = new Configuration(); conf.setBoolean(DFSConfigKeys.DFS_WEBHDFS_ENABLED_KEY, true); try { Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobInProgress.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobInProgress.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobInProgress.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobInProgress.java Wed Jul 4 17:56:51 2012 @@ -31,8 +31,8 @@ public class TestJobInProgress extends T private MiniDFSCluster dfsCluster; JobTracker jt; - private static Path TEST_DIR = - new Path(System.getProperty("test.build.data","/tmp"), "jip-testing"); + private static Path TEST_DIR = + new Path("/tmp/TestJobInProgress", "jip-testing"); private static int numSlaves = 4; public static class FailMapTaskJob extends MapReduceBase implements Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobQueueInformation.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobQueueInformation.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobQueueInformation.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobQueueInformation.java Wed Jul 4 17:56:51 2012 @@ -46,8 +46,7 @@ public class TestJobQueueInformation ext private JobConf jc; private static final String JOB_SCHEDULING_INFO = "TESTSCHEDULINGINFO"; private static final Path TEST_DIR = - new Path(System.getProperty("test.build.data","/tmp"), - "job-queue-info-testing"); + new Path("/tmp/TestJobQueueInformation", "job-queue-info-testing"); private static final Path IN_DIR = new Path(TEST_DIR, "input"); private static final Path SHARE_DIR = new Path(TEST_DIR, "share"); private static final Path OUTPUT_DIR = new Path(TEST_DIR, "output"); Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestSubmitJob.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestSubmitJob.java?rev=1357351&r1=1357350&r2=1357351&view=diff ============================================================================== --- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestSubmitJob.java (original) +++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestSubmitJob.java Wed Jul 4 17:56:51 2012 @@ -47,8 +47,7 @@ public class TestSubmitJob extends TestC private JobTracker jt; private FileSystem fs; private static Path TEST_DIR = - new Path(System.getProperty("test.build.data","/tmp"), - "job-submission-testing"); + new Path("/tmp/TestSubmitJob", "job-submission-testing"); private static int numSlaves = 1; private void startCluster() throws Exception {