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 ra...@apache.org on 2008/08/15 22:48:00 UTC

svn commit: r686362 - in /hadoop/core/trunk: CHANGES.txt src/hdfs/org/apache/hadoop/hdfs/server/datanode/BlockSender.java

Author: rangadi
Date: Fri Aug 15 13:48:00 2008
New Revision: 686362

URL: http://svn.apache.org/viewvc?rev=686362&view=rev
Log:
HADOOP-3933. DataNode sometimes sends up to io.byte.per.checksum bytes 
more than required to client. (Ning Li via rangadi)

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/datanode/BlockSender.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=686362&r1=686361&r2=686362&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Fri Aug 15 13:48:00 2008
@@ -324,6 +324,9 @@
 
     HADOOP-3949. Remove duplicate jars from Chukwa. (Jerome Boulon via omalley)
 
+    HADOOP-3933. DataNode sometimes sends up to io.byte.per.checksum bytes 
+    more than required to client. (Ning Li via rangadi)
+
 Release 0.18.0 - 2008-08-19
 
   INCOMPATIBLE CHANGES

Modified: hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/datanode/BlockSender.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/datanode/BlockSender.java?rev=686362&r1=686361&r2=686362&view=diff
==============================================================================
--- hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/datanode/BlockSender.java (original)
+++ hadoop/core/trunk/src/hdfs/org/apache/hadoop/hdfs/server/datanode/BlockSender.java Fri Aug 15 13:48:00 2008
@@ -133,7 +133,7 @@
       offset = (startOffset - (startOffset % bytesPerChecksum));
       if (length >= 0) {
         // Make sure endOffset points to end of a checksumed chunk.
-        long tmpLen = startOffset + length + (startOffset - offset);
+        long tmpLen = startOffset + length;
         if (tmpLen % bytesPerChecksum != 0) {
           tmpLen += (bytesPerChecksum - tmpLen % bytesPerChecksum);
         }