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);