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 dh...@apache.org on 2007/11/16 06:48:27 UTC
svn commit: r595563 - in /lucene/hadoop/trunk: CHANGES.txt
src/java/org/apache/hadoop/dfs/FSNamesystem.java
Author: dhruba
Date: Thu Nov 15 21:48:18 2007
New Revision: 595563
URL: http://svn.apache.org/viewvc?rev=595563&view=rev
Log:
HADOOP-2078. An zero size file may have no blocks associated with it.
(Konstantin Shvachko via dhruba)
Modified:
lucene/hadoop/trunk/CHANGES.txt
lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java
Modified: lucene/hadoop/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?rev=595563&r1=595562&r2=595563&view=diff
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Thu Nov 15 21:48:18 2007
@@ -104,6 +104,9 @@
to get created, thus causing unit test failure.
(Raghu Angadi via dhruba)
+ HADOOP-2078. An zero size file may have no blocks associated with it.
+ (Konstantin Shvachko via dhruba)
+
Release 0.15.1 -
IMPROVEMENTS
Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java?rev=595563&r1=595562&r2=595563&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java Thu Nov 15 21:48:18 2007
@@ -1076,13 +1076,10 @@
pendingFile = (INodeFileUnderConstruction) iFile;
fileBlocks = dir.getFileBlocks(src);
}
- if (fileBlocks == null || fileBlocks.length == 0 ||
- pendingFile == null) {
+ if (fileBlocks == null ) {
NameNode.stateChangeLog.warn("DIR* NameSystem.completeFile: "
+ "failed to complete " + src
- + " because dir.getFileBlocks() is " +
- ((fileBlocks == null) ?
- "null":fileBlocks.length) +
+ + " because dir.getFileBlocks() is null " +
" and pendingFile is " +
((pendingFile == null) ? "null" :
("from " + pendingFile.getClientMachine()))