Author: jimk
Date: Wed Aug 29 13:06:41 2007
New Revision: 570918

URL: http://svn.apache.org/viewvc?rev=570918&view=rev
Log:
HADOOP-1805 Region server hang on exit

Catch runtime exceptions in HMemcacheScanner constructor to ensure that read 
lock is released.

Modified:
    lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt
    
lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HMemcache.java

Modified: lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt?rev=570918&r1=570917&r2=570918&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt (original)
+++ lucene/hadoop/trunk/src/contrib/hbase/CHANGES.txt Wed Aug 29 13:06:41 2007
@@ -23,6 +23,7 @@
     HADOOP-1780 Regions are still being doubly assigned
     HADOOP-1797 Fix NPEs in MetaScanner constructor
     HADOOP-1799 Incorrect classpath in binary version of Hadoop
+    HADOOP-1805 Region server hang on exit
 
   IMPROVEMENTS
     HADOOP-1737 Make HColumnDescriptor data publically members settable

Modified: 
lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HMemcache.java
URL: 
http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HMemcache.java?rev=570918&r1=570917&r2=570918&view=diff
==============================================================================
--- 
lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HMemcache.java
 (original)
+++ 
lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HMemcache.java
 Wed Aug 29 13:06:41 2007
@@ -332,8 +332,13 @@
             }
           }
         }
+      } catch (RuntimeException ex) {
+        LOG.error("error initializing HMemcache scanner: ", ex);
+        close();
+        throw ex;
+        
       } catch(IOException ex) {
-        LOG.error(ex);
+        LOG.error("error initializing HMemcache scanner: ", ex);
         close();
         throw ex;
       }


Reply via email to