svn commit: r1042886 - /hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java

[stack]

Author: stack
Date: Tue Dec  7 01:19:41 2010
New Revision: 1042886

URL: http://svn.apache.org/viewvc?rev=1042886&view=rev
Log:
Fix possible NPE in assign

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

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=1042886&r1=1042885&r2=1042886&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
 Tue Dec  7 01:19:41 2010
@@ -956,7 +956,7 @@ public class AssignmentManager extends Z
     synchronized (this.regionPlans) {
       existingPlan = this.regionPlans.get(encodedName);
       if (existingPlan == null || forceNewPlan ||
-          existingPlan.getDestination().equals(serverToExclude)) {
+          (existingPlan != null && 
existingPlan.getDestination().equals(serverToExclude))) {
         newPlan = true;
         this.regionPlans.put(encodedName, randomPlan);
       }