Author: todd
Date: Thu Sep  1 21:13:21 2011
New Revision: 1164276

URL: http://svn.apache.org/viewvc?rev=1164276&view=rev
Log:
HBASE-4323  Add debug logging when AssignmentManager can't make a plan for a 
region.

Modified:
    hbase/branches/0.90/CHANGES.txt
    
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java

Modified: hbase/branches/0.90/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/branches/0.90/CHANGES.txt?rev=1164276&r1=1164275&r2=1164276&view=diff
==============================================================================
--- hbase/branches/0.90/CHANGES.txt (original)
+++ hbase/branches/0.90/CHANGES.txt Thu Sep  1 21:13:21 2011
@@ -38,6 +38,8 @@ Release 0.90.5 - Unreleased
                cases (todd)
    HBASE-4276  AssignmentManager debug logs should be at INFO level for
                META/ROOT regions (todd)
+   HBASE-4323  Add debug logging when AssignmentManager can't make a plan
+               for a region (todd)
  
 Release 0.90.4 - August 10, 2011
   BUG FIXES

Modified: 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
URL: 
http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java?rev=1164276&r1=1164275&r2=1164276&view=diff
==============================================================================
--- 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
 (original)
+++ 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
 Thu Sep  1 21:13:21 2011
@@ -1039,7 +1039,11 @@ public class AssignmentManager extends Z
         return;
       }
       RegionPlan plan = getRegionPlan(state, forceNewPlan);
-      if (plan == null) return; // Should get reassigned later when RIT times 
out.
+      if (plan == null) {
+        debugLog(state.getRegion(),
+            "Unable to determine a plan to assign " + state);
+        return; // Should get reassigned later when RIT times out.
+      }
       try {
         debugLog(state.getRegion(),
           "Assigning region " + state.getRegion().getRegionNameAsString() +


Reply via email to