You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jd...@apache.org on 2013/07/19 20:24:05 UTC
svn commit: r1504956 -
/hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
Author: jdcryans
Date: Fri Jul 19 18:24:04 2013
New Revision: 1504956
URL: http://svn.apache.org/r1504956
Log:
HBASE-8975 NPE/HTTP 500 when opening the master's web UI too early
Modified:
hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
Modified: hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java?rev=1504956&r1=1504955&r2=1504956&view=diff
==============================================================================
--- hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java (original)
+++ hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java Fri Jul 19 18:24:04 2013
@@ -63,7 +63,11 @@ public class MasterStatusServlet extends
List<ServerName> servers = null;
Set<ServerName> deadServers = null;
- if(master.isActiveMaster()){
+ if(master.isActiveMaster()) {
+ if (master.getServerManager() == null) {
+ response.sendError(503, "Master not ready");
+ return;
+ }
metaLocation = getMetaLocationOrNull(master);
//ServerName metaLocation = master.getCatalogTracker().getMetaLocation();
servers = master.getServerManager().getOnlineServersList();