svn commit: r908628 - in /hadoop/hdfs/trunk: CHANGES.txt src/java/org/apache/hadoop/hdfs/protocol/DataTransferProtocol.java

[szetszwo]

Author: szetszwo
Date: Wed Feb 10 18:52:24 2010
New Revision: 908628

URL: http://svn.apache.org/viewvc?rev=908628&view=rev
Log:
HDFS-930. Better error message for DATA_TRANSFER_VERSION mismatched.  
Contributed by Kay Kay

Modified:
    hadoop/hdfs/trunk/CHANGES.txt
    
hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/protocol/DataTransferProtocol.java

Modified: hadoop/hdfs/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/hdfs/trunk/CHANGES.txt?rev=908628&r1=908627&r2=908628&view=diff
==============================================================================
--- hadoop/hdfs/trunk/CHANGES.txt (original)
+++ hadoop/hdfs/trunk/CHANGES.txt Wed Feb 10 18:52:24 2010
@@ -63,6 +63,9 @@
     HDFS-949. Move DelegationToken into Common so that it can be used by
     MapReduce. (omalley)
 
+    HDFS-930. Better error message for DATA_TRANSFER_VERSION mismatched.
+    (Kay Kay via szetszwo)
+
   OPTIMIZATIONS
 
   BUG FIXES

Modified: 
hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/protocol/DataTransferProtocol.java
URL: 
http://svn.apache.org/viewvc/hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/protocol/DataTransferProtocol.java?rev=908628&r1=908627&r2=908628&view=diff
==============================================================================
--- 
hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/protocol/DataTransferProtocol.java
 (original)
+++ 
hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/protocol/DataTransferProtocol.java
 Wed Feb 10 18:52:24 2010
@@ -307,7 +307,9 @@
     protected final Op readOp(DataInputStream in) throws IOException {
       final short version = in.readShort();
       if (version != DATA_TRANSFER_VERSION) {
-        throw new IOException( "Version Mismatch" );
+        throw new IOException( "Version Mismatch (Expected: " +
+            DataTransferProtocol.DATA_TRANSFER_VERSION  +
+            ", Received: " +  version + " )");
       }
       return Op.read(in);
     }