ConfX created HBASE-27993:
-----------------------------

             Summary: AbstractFSWAL causes ArithmeticException due to improper 
logRollSize value checking
                 Key: HBASE-27993
                 URL: https://issues.apache.org/jira/browse/HBASE-27993
             Project: HBase
          Issue Type: Bug
            Reporter: ConfX
         Attachments: reproduce.sh

h2. What happened

There is no value checking for parameter 
{{hbase.regionserver.logroll.multiplier}} when {{hbase.wal.provider}} is set to 
{{{}multiwal{}}}. This may cause improper calculations and crashes the system 
like division by 0.
h2. Buggy code

In {{{}AbstractFSWAL.java{}}}, there is no value checking for {{logRollSize }} 
and this variable is directly used while calculating the value of maxLogFiles 
in the method {{{}calculateMaxLogFiles{}}}. When {{logRollSize }} is mistakenly 
set to 0, the code would cause division by 0 and throw ArithmeticException to 
crash the system.
{noformat}
     private int calculateMaxLogFiles(Configuration conf, long logRollSize) {
    Pair<Long, MemoryType> globalMemstoreSize = 
MemorySizeUtil.getGlobalMemStoreSize(conf);
    return (int) ((globalMemstoreSize.getFirst() * 2) / logRollSize);
  }{noformat}
h2. How to reproduce

(1) set hbase.regionserver.logroll.multiplier=0.0 and 
hbase.wal.provider=multiwal
(2) run org.apache.hadoop.hbase.wal.TestWALMethods#testGetSplitEditFilesSorted
you should observe the following failure:
{noformat}
java.lang.ArithmeticException: / by zero
        at 
org.apache.hadoop.hbase.regionserver.wal.AbstractFSWAL.calculateMaxLogFiles(AbstractFSWAL.java:466)
  at 
org.apache.hadoop.hbase.regionserver.wal.AbstractFSWAL.<init>(AbstractFSWAL.java:594)
        at 
org.apache.hadoop.hbase.regionserver.wal.AsyncFSWAL.<init>(AsyncFSWAL.java:141) 
       . . .{noformat}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to