svn commit: r780045 - in /hadoop/core/trunk: CHANGES.txt src/hdfs/org/apache/hadoop/hdfs/server/namenode/INode.java src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestINode.java

[szetszwo]

Author: szetszwo
Date: Fri May 29 17:52:30 2009
New Revision: 780045

URL: http://svn.apache.org/viewvc?rev=780045&view=rev
Log:
Revert HADOOP-5700.

Removed:
    
hadoop/core/trunk/src/test/hdfs/org/apache/hadoop/hdfs/server/namenode/TestINode.java
Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/namenode/INode.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=780045&r1=780044&r2=780045&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Fri May 29 17:52:30 2009
@@ -708,9 +708,6 @@
     HADOOP-5635. Change distributed cache to work with other distributed file
     systems. (Andrew Hitchcock via tomwhite)
 
-    HADOOP-5700. INode.getPathComponents throws NPE when given a non-absolute
-    path. (Todd Lipcon via tomwhite)
-
     HADOOP-5856. Fix "unsafe multithreaded use of DateFormat" findbugs warning
     in DataBlockScanner.  (Kan Zhang via szetszwo)
 

Modified: 
hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/namenode/INode.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/namenode/INode.java?rev=780045&r1=780044&r2=780045&view=diff
==============================================================================
--- 
hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/namenode/INode.java 
(original)
+++ 
hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/namenode/INode.java 
Fri May 29 17:52:30 2009
@@ -309,15 +309,11 @@
 
   /**
    * Breaks file path into components.
-   * @param path an absolute path on the file system
+   * @param path
    * @return array of byte arrays each of which represents 
    * a single path component.
    */
   static byte[][] getPathComponents(String path) {
-    if (!path.startsWith("/")) {
-      throw new IllegalArgumentException(
-        "Must pass an absolute path to getPathComponents");
-    }
     return getPathComponents(getPathNames(path));
   }