hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@apache.org
Subject svn commit: r938257 - in /hadoop/hbase/branches/0.20: CHANGES.txt src/java/org/apache/hadoop/hbase/master/ServerManager.java
Date Mon, 26 Apr 2010 22:05:12 GMT
Author: stack
Date: Mon Apr 26 22:05:11 2010
New Revision: 938257

URL: http://svn.apache.org/viewvc?rev=938257&view=rev
Log:
HBASE-2488 Master should warn more loudly about unexpected events

Modified:
    hadoop/hbase/branches/0.20/CHANGES.txt
    hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java

Modified: hadoop/hbase/branches/0.20/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/CHANGES.txt?rev=938257&r1=938256&r2=938257&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.20/CHANGES.txt Mon Apr 26 22:05:11 2010
@@ -156,6 +156,8 @@ Release 0.20.4 - Unreleased
    HBASE-2458  Client stuck in TreeMap,remove (Todd Lipcon via Stack)
    HBASE-2460  add_table.rb deletes any tables for which the target table name
                is a prefix (Todd Lipcon via Stack)
+   HBASE-2488  Master should warn more loudly about unexpected events
+               (Todd Lipcon via Stack)
 
   NEW FEATURES
    HBASE-2257  [stargate] multiuser mode

Modified: hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java?rev=938257&r1=938256&r2=938257&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java
(original)
+++ hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java
Mon Apr 26 22:05:11 2010
@@ -249,6 +249,8 @@ class ServerManager implements HConstant
   throws IOException {
     HServerInfo info = new HServerInfo(serverInfo);
     if (isDead(info.getServerName())) {
+      LOG.info("Received report from region server " + info.getServerName() +
+        " previously marked dead. Rejecting report.");
       throw new Leases.LeaseStillHeldException(info.getServerName());
     }
     if (msgs.length > 0) {
@@ -582,10 +584,8 @@ class ServerManager implements HConstant
       }
     
       if (duplicateAssignment) {
-        if (LOG.isDebugEnabled()) {
-          LOG.debug("region server " + serverInfo.getServerAddress().toString()
-              + " should not have opened region " + Bytes.toString(region.getRegionName()));
-        }
+        LOG.warn("region server " + serverInfo.getServerAddress().toString()
+            + " should not have opened region " + Bytes.toString(region.getRegionName()));
 
         // This Region should not have been opened.
         // Ask the server to shut it down, but don't report it as closed.  



Mime
View raw message