You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2022/06/17 00:20:10 UTC

[hbase] branch branch-2.5 updated: HBASE-27117 Update the method comments for RegionServerAccounting (#4532)

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

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


The following commit(s) were added to refs/heads/branch-2.5 by this push:
     new fa1b3eed99 HBASE-27117 Update the method comments for RegionServerAccounting (#4532)
fa1b3eed99 is described below

commit fa1b3eed99a4985784c2eb49d2584d14ede48a18
Author: litao <to...@gmail.com>
AuthorDate: Fri Jun 17 08:18:18 2022 +0800

    HBASE-27117 Update the method comments for RegionServerAccounting (#4532)
---
 .../java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java | 4 ++--
 .../apache/hadoop/hbase/regionserver/RegionServerAccounting.java   | 7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)

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 a268f2033f..44f0b769f1 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
@@ -428,14 +428,14 @@ public class MemStoreFlusher implements FlushRequester {
   }
 
   /**
-   * Return true if global memory usage is above the high watermark
+   * Return the FlushType if global memory usage is above the high watermark
    */
   private FlushType isAboveHighWaterMark() {
     return server.getRegionServerAccounting().isAboveHighWaterMark();
   }
 
   /**
-   * Return true if we're above the low watermark
+   * Return the FlushType if we're above the low watermark
    */
   private FlushType isAboveLowWaterMark() {
     return server.getRegionServerAccounting().isAboveLowWaterMark();
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerAccounting.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerAccounting.java
index bd9cdbe23b..fb13cd82ae 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerAccounting.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerAccounting.java
@@ -161,8 +161,8 @@ public class RegionServerAccounting {
   }
 
   /**
-   * Return true if we are above the memstore high water mark
-   * @return the flushtype
+   * Return the FlushType if we are above the memstore high water mark
+   * @return the FlushType
    */
   public FlushType isAboveHighWaterMark() {
     // for onheap memstore we check if the global memstore size and the
@@ -193,7 +193,8 @@ public class RegionServerAccounting {
   }
 
   /**
-   * Return true if we're above the low watermark
+   * Return the FlushType if we're above the low watermark
+   * @return the FlushType
    */
   public FlushType isAboveLowWaterMark() {
     // for onheap memstore we check if the global memstore size and the