You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2010/12/07 02:19:42 UTC
svn commit: r1042886 -
/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
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);
}