Skip to content

Commit e21f164

Browse files
tomscutvirajjasani
authored andcommitted
HBASE-27117 Update the method comments for RegionServerAccounting (#4532)
1 parent b2f71f2 commit e21f164

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -424,14 +424,14 @@ private boolean refreshStoreFilesAndReclaimMemory(Region region) {
424424
}
425425

426426
/**
427-
* Return true if global memory usage is above the high watermark
427+
* Return the FlushType if global memory usage is above the high watermark
428428
*/
429429
private FlushType isAboveHighWaterMark() {
430430
return server.getRegionServerAccounting().isAboveHighWaterMark();
431431
}
432432

433433
/**
434-
* Return true if we're above the low watermark
434+
* Return the FlushType if we're above the low watermark
435435
*/
436436
private FlushType isAboveLowWaterMark() {
437437
return server.getRegionServerAccounting().isAboveLowWaterMark();

hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerAccounting.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,8 +153,8 @@ public void decGlobalMemStoreSize(long dataSizeDelta, long heapSizeDelta, long o
153153
}
154154

155155
/**
156-
* Return true if we are above the memstore high water mark
157-
* @return the flushtype
156+
* Return the FlushType if we are above the memstore high water mark
157+
* @return the FlushType
158158
*/
159159
public FlushType isAboveHighWaterMark() {
160160
// for onheap memstore we check if the global memstore size and the
@@ -185,7 +185,8 @@ public FlushType isAboveHighWaterMark() {
185185
}
186186

187187
/**
188-
* Return true if we're above the low watermark
188+
* Return the FlushType if we're above the low watermark
189+
* @return the FlushType
189190
*/
190191
public FlushType isAboveLowWaterMark() {
191192
// for onheap memstore we check if the global memstore size and the

0 commit comments

Comments
 (0)