svn commit: r1028131 - /incubator/thrift/trunk/lib/java/src/org/apache/thrift/transport/TTransport.java

[bryanduxbury]

Author: bryanduxbury
Date: Wed Oct 27 23:15:35 2010
New Revision: 1028131

URL: http://svn.apache.org/viewvc?rev=1028131&view=rev
Log:
THRIFT-517. java:TExceptions thrown by server result in cryptic error message 
on client - Tried to read 4 bytes, but only got 0 bytes

This tiny change improves the message to make it marginally more obvious that 
the server-side might be the culprit.

Modified:
    
incubator/thrift/trunk/lib/java/src/org/apache/thrift/transport/TTransport.java

Modified: 
incubator/thrift/trunk/lib/java/src/org/apache/thrift/transport/TTransport.java
URL: 
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/java/src/org/apache/thrift/transport/TTransport.java?rev=1028131&r1=1028130&r2=1028131&view=diff
==============================================================================
--- 
incubator/thrift/trunk/lib/java/src/org/apache/thrift/transport/TTransport.java 
(original)
+++ 
incubator/thrift/trunk/lib/java/src/org/apache/thrift/transport/TTransport.java 
Wed Oct 27 23:15:35 2010
@@ -83,7 +83,12 @@ public abstract class TTransport {
     while (got < len) {
       ret = read(buf, off+got, len-got);
       if (ret <= 0) {
-        throw new TTransportException("Cannot read. Remote side has closed. 
Tried to read " + len + " bytes, but only got " + got + " bytes.");
+        throw new TTransportException(
+            "Cannot read. Remote side has closed. Tried to read "
+                + len
+                + " bytes, but only got "
+                + got
+                + " bytes. (This is often indicative of an internal error on 
the server side. Please check your server logs.)");
       }
       got += ret;
     }