You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2023/05/04 15:03:47 UTC

[hbase] 02/02: HBASE-27824 NPE in MetricsMasterWrapperImpl.isRunning (#5218)

This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git

commit 2eee1cc7e4db65e6b7ff1a1846d7d900316ebbae
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Thu May 4 20:59:40 2023 +0800

    HBASE-27824 NPE in MetricsMasterWrapperImpl.isRunning (#5218)
    
    Signed-off-by: Liangjun He <he...@apache.org>
    (cherry picked from commit 78610decaa5b6b5eca8c770e0eb33f73f27b83a9)
---
 .../java/org/apache/hadoop/hbase/master/MockNoopMasterServices.java | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/MockNoopMasterServices.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/MockNoopMasterServices.java
index 7d9f0cf4707..062e40aeeab 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/MockNoopMasterServices.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/MockNoopMasterServices.java
@@ -64,13 +64,9 @@ public class MockNoopMasterServices implements MasterServices {
   private final Configuration conf;
   private final MetricsMaster metricsMaster;
 
-  public MockNoopMasterServices() {
-    this(null);
-  }
-
   public MockNoopMasterServices(final Configuration conf) {
     this.conf = conf;
-    this.metricsMaster = new MetricsMaster(new MetricsMasterWrapperImpl(null));
+    this.metricsMaster = new MetricsMaster(new MetricsMasterWrapperImpl(mock(HMaster.class)));
   }
 
   @Override