You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cu...@apache.org on 2006/03/29 00:57:47 UTC
svn commit: r389620 - in /lucene/hadoop/trunk/src:
java/org/apache/hadoop/dfs/DistributedFileSystem.java
test/org/apache/hadoop/dfs/TestLocalDFS.java
Author: cutting
Date: Tue Mar 28 14:57:45 2006
New Revision: 389620
URL: http://svn.apache.org/viewcvs?rev=389620&view=rev
Log:
Fix unit tests on Windows. Don't assume that, just because a pathname begins with a slash that it also returns true for File.isAbsolute(). Instead use getAbsoluteFile() to force such things to be absolute.
Modified:
lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DistributedFileSystem.java
lucene/hadoop/trunk/src/test/org/apache/hadoop/dfs/TestLocalDFS.java
Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DistributedFileSystem.java
URL: http://svn.apache.org/viewcvs/lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DistributedFileSystem.java?rev=389620&r1=389619&r2=389620&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DistributedFileSystem.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DistributedFileSystem.java Tue Mar 28 14:57:45 2006
@@ -33,7 +33,7 @@
*****************************************************************/
public class DistributedFileSystem extends FileSystem {
private File workingDir =
- new File("/user/", System.getProperty("user.name"));
+ new File("/user", System.getProperty("user.name")).getAbsoluteFile();
private String name;
@@ -57,7 +57,7 @@
if (f.isAbsolute()) {
return f;
} else {
- return new File(workingDir, f.toString());
+ return new File(workingDir, f.getPath());
}
}
Modified: lucene/hadoop/trunk/src/test/org/apache/hadoop/dfs/TestLocalDFS.java
URL: http://svn.apache.org/viewcvs/lucene/hadoop/trunk/src/test/org/apache/hadoop/dfs/TestLocalDFS.java?rev=389620&r1=389619&r2=389620&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/test/org/apache/hadoop/dfs/TestLocalDFS.java (original)
+++ lucene/hadoop/trunk/src/test/org/apache/hadoop/dfs/TestLocalDFS.java Tue Mar 28 14:57:45 2006
@@ -46,7 +46,7 @@
writeFile(fileSys, file1);
assertTrue(fileSys.exists(new File(orig_path, file1.getPath())));
fileSys.delete(file1);
- File subdir1 = new File("/somewhere");
+ File subdir1 = new File("/somewhere").getAbsoluteFile();
fileSys.setWorkingDirectory(subdir1);
writeFile(fileSys, file1);
cleanupFile(fileSys, new File(subdir1, file1.getPath()));