You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jx...@apache.org on 2013/08/18 17:42:30 UTC

svn commit: r1515147 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java

Author: jxiang
Date: Sun Aug 18 15:42:30 2013
New Revision: 1515147

URL: http://svn.apache.org/r1515147
Log:
HBASE-9256 HBaseClient#setupIOStreams should handle all exceptions

Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java?rev=1515147&r1=1515146&r2=1515147&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java Sun Aug 18 15:42:30 2013
@@ -445,8 +445,14 @@ public class HBaseClient {
 
         // start the receiver thread after the socket connection has been set up
         start();
-      } catch (IOException e) {
+      } catch (Throwable t) {
         failedServers.addToFailedServers(remoteId.address);
+        IOException e;
+        if (t instanceof IOException) {
+          e = (IOException)t;
+        } else {
+          e = new IOException("Could not set up IO Streams", t);
+        }
         markClosed(e);
         close();