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 cd...@apache.org on 2008/04/02 02:44:43 UTC
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
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
*/