hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cutt...@apache.org
Subject svn commit: r391425 - in /lucene/hadoop/trunk: CHANGES.txt src/java/org/apache/hadoop/mapred/JobConf.java
Date Tue, 04 Apr 2006 22:04:48 GMT
Author: cutting
Date: Tue Apr  4 15:04:47 2006
New Revision: 391425

URL: http://svn.apache.org/viewcvs?rev=391425&view=rev
Log:
Fix HADOOP-117.  Correctly remove mapred temp files.

Modified:
    lucene/hadoop/trunk/CHANGES.txt
    lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobConf.java

Modified: lucene/hadoop/trunk/CHANGES.txt
URL: http://svn.apache.org/viewcvs/lucene/hadoop/trunk/CHANGES.txt?rev=391425&r1=391424&r2=391425&view=diff
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Tue Apr  4 15:04:47 2006
@@ -8,6 +8,10 @@
     Closeable interface.  This permits subclasses which override this
     method to throw that exception. (cutting)
 
+ 3. Fix HADOOP-117.  Pathnames were mistakenly transposed in
+    JobConf.getLocalFile() causing many mapred temporary files to not
+    be removed.  (Raghavendra Prabhu via cutting)
+
 
 Release 0.1.0 - 2006-04-01
 

Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobConf.java
URL: http://svn.apache.org/viewcvs/lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobConf.java?rev=391425&r1=391424&r2=391425&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobConf.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobConf.java Tue Apr  4 15:04:47
2006
@@ -124,7 +124,7 @@
   /** Constructs a local file name.  Files are distributed among configured
    * local directories.*/
   public File getLocalFile(String subdir, String name) throws IOException {
-    return getFile("mapred.local.dir", name + File.separator + subdir);
+    return getFile("mapred.local.dir", subdir + File.separator + name);
   }
 
   public void setInputDir(File dir) { set("mapred.input.dir", dir); }



Mime
View raw message