svn commit: r931724 - in /hadoop/hbase/branches/0.20_pre_durability: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/HRegion.java src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

[stack]

Author: stack
Date: Wed Apr  7 23:47:35 2010
New Revision: 931724

URL: http://svn.apache.org/viewvc?rev=931724&view=rev
Log:
HBASE-2419 Remove from RS logs the fat NotServingRegionException stack

Modified:
    hadoop/hbase/branches/0.20_pre_durability/CHANGES.txt
    
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java
    
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Modified: hadoop/hbase/branches/0.20_pre_durability/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20_pre_durability/CHANGES.txt?rev=931724&r1=931723&r2=931724&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20_pre_durability/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.20_pre_durability/CHANGES.txt Wed Apr  7 23:47:35 
2010
@@ -55,6 +55,7 @@ Release 0.20.4 - Unreleased
                holds up all flushing
    HBASE-2252  Mapping a very big table kills region servers
    HBASE-2277  Update branch to hadoop 0.20.2
+   HBASE-2419  Remove from RS logs the fat NotServingRegionException stack
 
   NEW FEATURES
    HBASE-2257  [stargate] multiuser mode

Modified: 
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java
URL: 
http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java?rev=931724&r1=931723&r2=931724&view=diff
==============================================================================
--- 
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java
 (original)
+++ 
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java
 Wed Apr  7 23:47:35 2010
@@ -1623,7 +1623,7 @@ public class HRegion implements HConstan
     splitsAndClosesLock.readLock().lock();
     try {
       if (this.closed.get()) {
-        throw new NotServingRegionException("Region " + this + " closed");
+        throw new NotServingRegionException(this + " is closed");
       }
       Integer key = Bytes.mapKey(row);
       synchronized (locksToRows) {

Modified: 
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: 
http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=931724&r1=931723&r2=931724&view=diff
==============================================================================
--- 
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
 (original)
+++ 
hadoop/hbase/branches/0.20_pre_durability/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
 Wed Apr  7 23:47:35 2010
@@ -1503,7 +1503,7 @@ public class HRegionServer implements HC
                   + e.msg.toString());
             }
           } catch (InterruptedException ex) {
-            // continue
+            LOG.warn("Processing Worker queue", ex);
           } catch (Exception ex) {
             if (ex instanceof IOException) {
               ex = RemoteExceptionHandler.checkIOException((IOException) ex);
@@ -1794,7 +1794,8 @@ public class HRegionServer implements HC
         region.put(puts[i], locks[i]);
       }
     } catch (WrongRegionException ex) {
-      LOG.debug("Batch puts: " + i, ex);
+      LOG.debug("Batch puts interrupted at index=" + i + " because:" +
+        ex.getMessage());
       return i;
     } catch (NotServingRegionException ex) {
       return i;
@@ -2511,5 +2512,4 @@ public class HRegionServer implements HC
         HRegionServer.class);
     doMain(args, regionServerClass);
   }
-
 }