You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-commits@hadoop.apache.org by to...@apache.org on 2011/06/15 23:38:45 UTC
svn commit: r1136209 - in /hadoop/common/branches/branch-0.22/hdfs:
CHANGES.txt src/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java
Author: todd
Date: Wed Jun 15 21:38:44 2011
New Revision: 1136209
URL: http://svn.apache.org/viewvc?rev=1136209&view=rev
Log:
HDFS-2071. Use of isConnected() in DataXceiver is invalid. Contributed by Kihwal Lee.
Modified:
hadoop/common/branches/branch-0.22/hdfs/CHANGES.txt
hadoop/common/branches/branch-0.22/hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java
Modified: hadoop/common/branches/branch-0.22/hdfs/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.22/hdfs/CHANGES.txt?rev=1136209&r1=1136208&r2=1136209&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.22/hdfs/CHANGES.txt (original)
+++ hadoop/common/branches/branch-0.22/hdfs/CHANGES.txt Wed Jun 15 21:38:44 2011
@@ -584,6 +584,9 @@ Release 0.22.0 - Unreleased
HDFS-988. saveNamespace race can corrupt the edits log. (eli)
+ HDFS-2071. Use of isConnected() in DataXceiver is invalid. (Kihwal Lee
+ via todd)
+
Release 0.21.1 - Unreleased
IMPROVEMENTS
Modified: hadoop/common/branches/branch-0.22/hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.22/hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java?rev=1136209&r1=1136208&r2=1136209&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.22/hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java (original)
+++ hadoop/common/branches/branch-0.22/hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java Wed Jun 15 21:38:44 2011
@@ -171,7 +171,7 @@ class DataXceiver extends DataTransferPr
opStartTime = now();
processOp(op, in);
++opsProcessed;
- } while (s.isConnected() && socketKeepaliveTimeout > 0);
+ } while (!s.isClosed() && socketKeepaliveTimeout > 0);
} catch (Throwable t) {
LOG.error(datanode.dnRegistration + ":DataXceiver, at " +
s.toString(), t);