You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by mb...@apache.org on 2012/04/21 03:30:56 UTC
svn commit: r1328558 -
/hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ProcessServerShutdown.java
Author: mbautin
Date: Sat Apr 21 01:30:55 2012
New Revision: 1328558
URL: http://svn.apache.org/viewvc?rev=1328558&view=rev
Log:
[master] npe fix
Summary: -
Test Plan: compiled
Reviewers: liyintang, aaiyer, kannan
Reviewed By: kannan
CC: hbase-eng@
Differential Revision: https://phabricator.fb.com/D450781
Task ID: 1015472
Modified:
hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ProcessServerShutdown.java
Modified: hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ProcessServerShutdown.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ProcessServerShutdown.java?rev=1328558&r1=1328557&r2=1328558&view=diff
==============================================================================
--- hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ProcessServerShutdown.java (original)
+++ hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ProcessServerShutdown.java Sat Apr 21 01:30:55 2012
@@ -250,8 +250,13 @@ class ProcessServerShutdown extends Regi
public Boolean call() throws IOException {
if (LOG.isDebugEnabled()) {
- LOG.debug("Process server shutdown scanning root region on " +
- master.getRegionManager().getRootRegionLocation().getBindAddress());
+ HServerAddress addr = master.getRegionManager().getRootRegionLocation();
+ if (addr != null) {
+ LOG.debug("Process server shutdown scanning root region on " +
+ addr.getBindAddress());
+ } else {
+ LOG.debug("ProcessServerShutdown scanning root, but root is null");
+ }
}
Scan scan = new Scan();
scan.addFamily(HConstants.CATALOG_FAMILY);