You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 19:43:00 UTC

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

Author: nspiegelberg
Date: Tue Oct 11 17:42:59 2011
New Revision: 1181936

URL: http://svn.apache.org/viewvc?rev=1181936&view=rev
Log:
fix stuck IPC client in HBase-trunk

Summary:
Port Hairong's fix in Hadoop-ipc into hbase trunk
https://phabricator.fb.com/D185014

Not sure this bug is the root cause of ipc dead lock in server
But we'd better port this fix to our trunk.

Test Plan: Running all the unit tests
Reviewed By: pkhemani
Reviewers: nspiegelberg, kranganathan, gqchen, mbautin, pritam, hkuang, jgray,
pkhemani
Commenters: jgray
CC: hbase@lists, , dhruba, jgray, pkhemani, liyintang
Differential Revision: 292784

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

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java?rev=1181936&r1=1181935&r2=1181936&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/ipc/HBaseClient.java Tue Oct 11 17:42:59 2011
@@ -537,6 +537,8 @@ public class HBaseClient {
         }
       } catch (IOException e) {
         markClosed(e);
+      } catch (Throwable te) {
+        markClosed((IOException)new IOException().initCause(te));
       }
     }