svn commit: r1040216 - in /hbase/trunk: CHANGES.txt src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

[jgray]

Author: jgray
Date: Mon Nov 29 18:20:11 2010
New Revision: 1040216

URL: http://svn.apache.org/viewvc?rev=1040216&view=rev
Log:
HBASE-3280  YouAreDeadException being swallowed in HRS getMaster

Modified:
    hbase/trunk/CHANGES.txt
    
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Modified: hbase/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1040216&r1=1040215&r2=1040216&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Mon Nov 29 18:20:11 2010
@@ -6,6 +6,7 @@ Release 0.91.0 - Unreleased
                changes at runtime (Gary Helmling via Andrew Purtell)
 
   BUG FIXES
+   HBASE-3280  YouAreDeadException being swallowed in HRS getMaster
 
   IMPROVEMENTS
    HBASE-2001  Coprocessors: Colocate user code with regions (Mingjie Lai via

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=1040216&r1=1040215&r2=1040216&view=diff
==============================================================================
--- 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
 (original)
+++ 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
 Mon Nov 29 18:20:11 2010
@@ -686,6 +686,13 @@ public class HRegionServer implements HR
           getMostLoadedRegions());
         break;
       } catch (IOException ioe) {
+        if (ioe instanceof RemoteException) {
+          ioe = ((RemoteException)ioe).unwrapRemoteException();
+        }
+        if (ioe instanceof YouAreDeadException) {
+          // This will be caught and handled as a fatal error in run()
+          throw ioe;
+        }
         // Couldn't connect to the master, get location from zk and reconnect
         // Method blocks until new master is found or we are stopped
         getMaster();