You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jx...@apache.org on 2013/07/27 00:57:41 UTC
svn commit: r1507502 -
/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
Author: jxiang
Date: Fri Jul 26 22:57:41 2013
New Revision: 1507502
URL: http://svn.apache.org/r1507502
Log:
HBASE-9048 HCM throws NullPointerException under load
Modified:
hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
Modified: hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java?rev=1507502&r1=1507501&r2=1507502&view=diff
==============================================================================
--- hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java (original)
+++ hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java Fri Jul 26 22:57:41 2013
@@ -1085,7 +1085,9 @@ public class HConnectionManager {
for (Map<byte[], HRegionLocation> tableLocations :
cachedRegionLocations.values()) {
for (Entry<byte[], HRegionLocation> e : tableLocations.entrySet()) {
- if (serverName.equals(e.getValue().getServerName())) {
+ HRegionLocation value = e.getValue();
+ if (value != null
+ && serverName.equals(value.getServerName())) {
tableLocations.remove(e.getKey());
deletedSomething = true;
}