liuxiaocs7 commented on code in PR #7726:
URL: https://github.com/apache/hbase/pull/7726#discussion_r2875890811
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/io/util/MemorySizeUtil.java:
##########
@@ -151,10 +154,29 @@ public static float
getRegionServerMinFreeHeapFraction(final Configuration conf)
/**
* Retrieve global memstore configured size as percentage of total heap.
*/
- public static float getGlobalMemStoreHeapPercent(final Configuration c,
+ public static float getGlobalMemStoreHeapPercent(final Configuration conf,
final boolean logInvalid) {
+ // Check if an explicit memstore size is configured.
+ long memStoreSizeInBytes = getMemstoreSizeInBytes(conf);
Review Comment:
> What happens if MEMSTORE_MEMORY_SIZE_KEY is not set. Won't this result in
spurious warnings?
if `MEMSTORE_MEMORY_SIZE_KEY` is not set, we will got `-1`, add a branch for
this scenario to print different warning logs, WDYT, thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
