Author: todd
Date: Thu Sep  1 21:14:08 2011
New Revision: 1164277

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

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

Modified: hbase/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1164277&r1=1164276&r2=1164277&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Thu Sep  1 21:14:08 2011
@@ -527,6 +527,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
 

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java?rev=1164277&r1=1164276&r2=1164277&view=diff
==============================================================================
--- 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 
(original)
+++ 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 
Thu Sep  1 21:14:08 2011
@@ -1236,7 +1236,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