Repository: hbase Updated Branches: refs/heads/master 8793383b6 -> 8b7a20f4e
HBASE-12393 The regionserver web will throw exception if we disable block cache (ChiaPing Tsai) Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8b7a20f4 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8b7a20f4 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8b7a20f4 Branch: refs/heads/master Commit: 8b7a20f4eefffe8002d1e1e31f9dde43da43e3ad Parents: 8793383 Author: Enis Soztutar <[email protected]> Authored: Wed Jan 21 17:22:10 2015 -0800 Committer: Enis Soztutar <[email protected]> Committed: Wed Jan 21 17:22:10 2015 -0800 ---------------------------------------------------------------------- .../hadoop/hbase/tmpl/regionserver/BlockCacheTmpl.jamon | 2 +- .../hadoop/hbase/regionserver/TestRSStatusServlet.java | 11 +++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/8b7a20f4/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/BlockCacheTmpl.jamon ---------------------------------------------------------------------- diff --git a/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/BlockCacheTmpl.jamon b/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/BlockCacheTmpl.jamon index a6a3cf9..e4ff70f 100644 --- a/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/BlockCacheTmpl.jamon +++ b/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/BlockCacheTmpl.jamon @@ -210,7 +210,7 @@ org.apache.hadoop.util.StringUtils; <%args> CacheConfig cacheConfig; </%args> -<%if cacheConfig == null %> +<%if cacheConfig == null || cacheConfig.getBlockCache() == null %> <p>CacheConfig is null</p> <%else> <table class="table table-striped"> http://git-wip-us.apache.org/repos/asf/hbase/blob/8b7a20f4/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRSStatusServlet.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRSStatusServlet.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRSStatusServlet.java index 22a3546..732df4b 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRSStatusServlet.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRSStatusServlet.java @@ -45,12 +45,17 @@ import org.mockito.Mockito; import com.google.common.collect.Lists; import com.google.protobuf.RpcController; import com.google.protobuf.ServiceException; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; +import org.apache.hadoop.hbase.HConstants; +import org.apache.hadoop.hbase.io.hfile.CacheConfig; /** * Tests for the region server status page and its template. */ @Category({RegionServerTests.class, SmallTests.class}) public class TestRSStatusServlet { + private static final Log LOG = LogFactory.getLog(TestRSStatusServlet.class); private HRegionServer rs; private RSRpcServices rpcServices; @@ -79,6 +84,12 @@ public class TestRSStatusServlet { Mockito.doReturn("fakequorum").when(zkw).getQuorum(); Mockito.doReturn(zkw).when(rs).getZooKeeper(); + // Fake CacheConfig + LOG.warn("The " + HConstants.HFILE_BLOCK_CACHE_SIZE_KEY + " is set to 0"); + CacheConfig cacheConf = Mockito.mock(CacheConfig.class); + Mockito.doReturn(null).when(cacheConf).getBlockCache(); + Mockito.doReturn(cacheConf).when(rs).getCacheConfig(); + // Fake MasterAddressTracker MasterAddressTracker mat = Mockito.mock(MasterAddressTracker.class); Mockito.doReturn(fakeMasterAddress).when(mat).getMasterAddress();
