Set special reason for timeout exceptions

mziccard · mziccard · commit a0f2d2064771 · 2015-11-24T19:01:55.000+01:00
diff --git a/gcloud-java-datastore/src/main/java/com/google/gcloud/spi/DefaultDatastoreRpc.java b/gcloud-java-datastore/src/main/java/com/google/gcloud/spi/DefaultDatastoreRpc.java
@@ -126,10 +126,12 @@ private static DatastoreRpcException translate(DatastoreException exception) {
       return new DatastoreRpcException(reason);
     } else {
       boolean retryable = false;
+      reasonStr = "Unknown";
       if (exception.getCause() instanceof SocketTimeoutException) {
         retryable = true;
+        reasonStr = "Request timeout";
       }
-      return new DatastoreRpcException("Unknown", exception.getCode(), retryable, message);
+      return new DatastoreRpcException(reasonStr, exception.getCode(), retryable, message);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -126,10 +126,12 @@ private static DatastoreRpcException translate(DatastoreException exception) {`
`126`	`126`	`return new DatastoreRpcException(reason);`
`127`	`127`	`} else {`
`128`	`128`	`boolean retryable = false;`
	`129`	`+ reasonStr = "Unknown";`
`129`	`130`	`if (exception.getCause() instanceof SocketTimeoutException) {`
`130`	`131`	`retryable = true;`
	`132`	`+ reasonStr = "Request timeout";`
`131`	`133`	`}`
`132`		`- return new DatastoreRpcException("Unknown", exception.getCode(), retryable, message);`
	`134`	`+ return new DatastoreRpcException(reasonStr, exception.getCode(), retryable, message);`
`133`	`135`	`}`
`134`	`136`	`}`
`135`	`137`