[hbase] branch master updated: HBASE-23702 hbase.hstore.flusher.count setting to 0 breaks HMaster (#1703)

[zg...@apache.org]

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

zghao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new bab4b1c  HBASE-23702 hbase.hstore.flusher.count setting to 0 breaks HMaster (#1703)
bab4b1c is described below

commit bab4b1cf8c57456794e15e556a7483b88c504ae2
Author: Guanghao Zhang <zg...@apache.org>
AuthorDate: Fri May 15 09:05:25 2020 +0800

    HBASE-23702 hbase.hstore.flusher.count setting to 0 breaks HMaster (#1703)
    
    Co-authored-by: Ctest
    Signed-off-by: Anoop Sam John <an...@gmail.com>
    Signed-off-by: Wellington Chevreuil <wc...@apache.org>
---
 .../java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java   | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
index b94afab..9f4d881 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
@@ -129,6 +129,11 @@ class MemStoreFlusher implements FlushRequester {
     this.blockingWaitTime = conf.getInt("hbase.hstore.blockingWaitTime",
       90000);
     int handlerCount = conf.getInt("hbase.hstore.flusher.count", 2);
+    if (handlerCount < 1) {
+      LOG.warn("hbase.hstore.flusher.count was configed to {} which is less than 1, corrected to 1",
+          handlerCount);
+      handlerCount = 1;
+    }
     this.flushHandlers = new FlushHandler[handlerCount];
     LOG.info("globalMemStoreLimit="
         + TraditionalBinaryPrefix