You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 19:44:31 UTC
svn commit: r1181963 -
/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java
Author: nspiegelberg
Date: Tue Oct 11 17:44:31 2011
New Revision: 1181963
URL: http://svn.apache.org/viewvc?rev=1181963&view=rev
Log:
fix null pointer exception
Test Plan:
mvn package
mvn test (running)
Reviewers: kannan, kranganathan
Reviewed By: kannan
CC: kannan
Differential Revision: 316203
Modified:
hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java
Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java?rev=1181963&r1=1181962&r2=1181963&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java Tue Oct 11 17:44:31 2011
@@ -601,10 +601,11 @@ public class RegionManager {
}
}
if (assignmentByLocality && !i.isRootRegion() && !i.isMetaRegion()) {
- String preferredHost =
- this.master.getPreferredRegionToRegionServerMapping().get(new Text(name)).toString();
+ Text preferredHostNameTxt =
+ (Text)this.master.getPreferredRegionToRegionServerMapping().get(new Text(name));
- if (preferredHost != null) {
+ if (preferredHostNameTxt != null) {
+ String preferredHost = preferredHostNameTxt.toString();
if (hostName.startsWith(preferredHost)) {
LOG.debug("Doing Preferred Region Assignment for : " + name +
" to the " + hostName);