hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cdoug...@apache.org
Subject svn commit: r643667 - in /hadoop/core/trunk: CHANGES.txt src/java/org/apache/hadoop/dfs/FSImage.java src/java/org/apache/hadoop/dfs/FSNamesystem.java src/java/org/apache/hadoop/dfs/INode.java
Date Wed, 02 Apr 2008 00:44:43 GMT
Author: cdouglas
Date: Tue Apr  1 17:44:36 2008
New Revision: 643667

URL: http://svn.apache.org/viewvc?rev=643667&view=rev
Log:
HADOOP-2831. Remove deprecated o.a.h.dfs.INode::getAbsoluteName().
(lohit vijayarenu via cdouglas)



Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSImage.java
    hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java
    hadoop/core/trunk/src/java/org/apache/hadoop/dfs/INode.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=643667&r1=643666&r2=643667&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Tue Apr  1 17:44:36 2008
@@ -68,6 +68,9 @@
     HADOOP-2818.  Remove deprecated methods in Counters.
     (Amareshwari Sriramadasu via tomwhite)
 
+    HADOOP-2831. Remove deprecated o.a.h.dfs.INode::getAbsoluteName()
+    (lohit vijayarenu via cdouglas)
+
   NEW FEATURES
 
     HADOOP-1398.  Add HBase in-memory block cache.  (tomwhite)

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSImage.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSImage.java?rev=643667&r1=643666&r2=643667&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSImage.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSImage.java Tue Apr  1 17:44:36 2008
@@ -981,9 +981,10 @@
   // into the input stream
   //
   static void writeINodeUnderConstruction(DataOutputStream out,
-                                           INodeFileUnderConstruction cons) 
+                                           INodeFileUnderConstruction cons,
+                                           String path) 
                                            throws IOException {
-    new UTF8(cons.getAbsoluteName()).write(out);
+    new UTF8(path).write(out);
     out.writeShort(cons.getReplication());
     out.writeLong(cons.getModificationTime());
     out.writeLong(cons.getPreferredBlockSize());

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java?rev=643667&r1=643666&r2=643667&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java Tue Apr  1 17:44:36
2008
@@ -4379,7 +4379,7 @@
                                   " but is not under construction.");
           }
           INodeFileUnderConstruction cons = (INodeFileUnderConstruction) node;
-          FSImage.writeINodeUnderConstruction(out, cons);
+          FSImage.writeINodeUnderConstruction(out, cons, path);
         }
       }
     }

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/dfs/INode.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/dfs/INode.java?rev=643667&r1=643666&r2=643667&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/dfs/INode.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/dfs/INode.java Tue Apr  1 17:44:36 2008
@@ -175,24 +175,6 @@
   }
 
   /**
-   * Get the full absolute path name of this file (recursively computed).
-   * 
-   * @return the string representation of the absolute path of this file
-   * 
-   * @deprecated this is used only in crc upgrade now, and should be removed 
-   * in order to be able to eliminate the parent field. 
-   */
-  String getAbsoluteName() {
-    if (this.parent == null) {
-      return Path.SEPARATOR;       // root directory is "/"
-    }
-    if (this.parent.parent == null) {
-      return Path.SEPARATOR + getLocalName();
-    }
-    return parent.getAbsoluteName() + Path.SEPARATOR + getLocalName();
-  }
-
-  /**
    * Get parent directory 
    * @return parent INode
    */



Mime
View raw message