hadoop-common-commits mailing list archives

Author: shv
Date: Tue Apr 22 18:21:53 2008
New Revision: 650715

URL: http://svn.apache.org/viewvc?rev=650715&view=rev
Log:
HADOOP-3272. Remove redundant copy of Block object in BlocksMap. Contributed by Lohit Vjayarenu.

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/java/org/apache/hadoop/dfs/BlocksMap.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=650715&r1=650714&r2=650715&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Tue Apr 22 18:21:53 2008
@@ -36,6 +36,9 @@
     HADOOP-3274. The default constructor of BytesWritable creates empty 
     byte array. (Tsz Wo (Nicholas), SZE via shv)
 
+    HADOOP-3272. Remove redundant copy of Block object in BlocksMap.
+    (Lohit Vjayarenu via shv)
+
   BUG FIXES
 
     HADOOP-2905. 'fsck -move' triggers NPE in NameNode. 

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/dfs/BlocksMap.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/dfs/BlocksMap.java?rev=650715&r1=650714&r2=650715&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/dfs/BlocksMap.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/dfs/BlocksMap.java Tue Apr 22 18:21:53 2008
@@ -297,7 +297,7 @@
     BlockInfo info = map.get(b);
     if (info == null) {
       info = new BlockInfo(b, replication);
-      map.put(b, info);
+      map.put(info, info);
     }
     return info;
   }