Author: tedyu
Date: Tue Aug 2 04:54:37 2011
New Revision: 1153027
URL: http://svn.apache.org/viewvc?rev=1153027&view=rev
Log:
Check hris against null in ServerShutdownHandler.process()
Modified:
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java
Modified:
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java
URL:
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java?rev=1153027&r1=1153026&r2=1153027&view=diff
==============================================================================
---
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java
(original)
+++
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java
Tue Aug 2 04:54:37 2011
@@ -220,11 +220,13 @@ public class ServerShutdownHandler exten
" regions(s) that are already in transition)");
// Iterate regions that were on this server and assign them
- for (Map.Entry<HRegionInfo, Result> e: hris.entrySet()) {
- if (processDeadRegion(e.getKey(), e.getValue(),
- this.services.getAssignmentManager(),
- this.server.getCatalogTracker())) {
- this.services.getAssignmentManager().assign(e.getKey(), true);
+ if (hris != null) {
+ for (Map.Entry<HRegionInfo, Result> e: hris.entrySet()) {
+ if (processDeadRegion(e.getKey(), e.getValue(),
+ this.services.getAssignmentManager(),
+ this.server.getCatalogTracker())) {
+ this.services.getAssignmentManager().assign(e.getKey(), true);
+ }
}
}
this.deadServers.finish(serverName);