svn commit: r723219 - in /hadoop/core/trunk: CHANGES.txt src/test/org/apache/hadoop/mapred/TestJobHistoryVersion.java

[cdouglas]

Author: cdouglas
Date: Wed Dec  3 22:06:05 2008
New Revision: 723219

URL: http://svn.apache.org/viewvc?rev=723219&view=rev
Log:
HADOOP-4632. Fix TestJobHistoryVersion to use test.build.dir instead of the
current workding directory for scratch space. Contributed by Amar Kamat.

Modified:
    hadoop/core/trunk/CHANGES.txt
    
hadoop/core/trunk/src/test/org/apache/hadoop/mapred/TestJobHistoryVersion.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=723219&r1=723218&r2=723219&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Wed Dec  3 22:06:05 2008
@@ -253,9 +253,6 @@
     HADOOP-4732. Pass connection and read timeouts in the correct order when
     setting up fetch in reduce. (Amareshwari Sriramadasu via cdouglas)
 
-    HADOOP-4420. Add null checks for job, caused by invalid job IDs.
-    (Aaron Kimball via tomwhite)
-
 Release 0.19.1 - Unreleased
 
   IMPROVEMENTS
@@ -268,6 +265,12 @@
     HADOOP-4697. Fix getBlockLocations in KosmosFileSystem to handle multiple
     blocks correctly. (Sriram Rao via cdouglas)
 
+    HADOOP-4420. Add null checks for job, caused by invalid job IDs.
+    (Aaron Kimball via tomwhite)
+
+    HADOOP-4632. Fix TestJobHistoryVersion to use test.build.dir instead of the
+    current workding directory for scratch space. (Amar Kamat via cdouglas)
+
 Release 0.19.0 - 2008-11-18
 
   INCOMPATIBLE CHANGES

Modified: 
hadoop/core/trunk/src/test/org/apache/hadoop/mapred/TestJobHistoryVersion.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/src/test/org/apache/hadoop/mapred/TestJobHistoryVersion.java?rev=723219&r1=723218&r2=723219&view=diff
==============================================================================
--- 
hadoop/core/trunk/src/test/org/apache/hadoop/mapred/TestJobHistoryVersion.java 
(original)
+++ 
hadoop/core/trunk/src/test/org/apache/hadoop/mapred/TestJobHistoryVersion.java 
Wed Dec  3 22:06:05 2008
@@ -43,7 +43,9 @@
     + "Map-Reduce Framework.Map input records:0,"
     + "Map-Reduce Framework.Map input bytes:0,"
     + "File Systems.HDFS bytes written:0,";
-  private static final String TEST_DIR = "test-history-version";
+  private static final Path TEST_DIR = 
+    new Path(System.getProperty("test.build.data", "/tmp"), 
+             "test-history-version");
   private static final String DELIM = ".";
   
   
@@ -115,6 +117,9 @@
     JobConf conf = new JobConf();
     FileSystem fs = FileSystem.getLocal(conf);
     
+    // cleanup
+    fs.delete(TEST_DIR, true);
+    
     Path historyPath = new Path(TEST_DIR + "/_logs/history/" + FILENAME);
     
     fs.delete(historyPath, false);
@@ -128,6 +133,9 @@
     
     assertTrue("Failed to parse old jobhistory files", 
                job.getAllTasks().size() > 0);
+    
+    // cleanup
+    fs.delete(TEST_DIR, true);
   }
   
   /**
@@ -137,6 +145,9 @@
     JobConf conf = new JobConf();
     FileSystem fs = FileSystem.getLocal(conf);
     
+    // cleanup
+    fs.delete(TEST_DIR, true);
+    
     Path historyPath = new Path(TEST_DIR + "/_logs/history/" + FILENAME);
     
     fs.delete(historyPath, false);
@@ -150,5 +161,8 @@
     
     assertTrue("Failed to parse old jobhistory files", 
                job.getAllTasks().size() > 0);
+    
+    // cleanup
+    fs.delete(TEST_DIR, true);
   }
 }