[hbase] branch branch-2.2 updated: HBASE-24165 maxPoolSize is logged incorrectly in ByteBufferPool (#1624)

[st...@apache.org]

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

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


The following commit(s) were added to refs/heads/branch-2.2 by this push:
     new 0b4f272  HBASE-24165 maxPoolSize is logged incorrectly in ByteBufferPool (#1624)
0b4f272 is described below

commit 0b4f272f2d4d9f9bd1d3d622b14d8bea13ab31ad
Author: Pankaj <pa...@apache.org>
AuthorDate: Wed May 13 05:03:58 2020 +0530

    HBASE-24165 maxPoolSize is logged incorrectly in ByteBufferPool (#1624)
    
    
    Signed-off-by: Jan Hentschel <ja...@apache.org>
    Signed-off-by: Viraj Jasani <vj...@apache.org>
    Signed-off-by: stack <st...@apache.org>
---
 .../src/main/java/org/apache/hadoop/hbase/io/ByteBufferPool.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/io/ByteBufferPool.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/io/ByteBufferPool.java
index caca20b..abd088a 100644
--- a/hbase-common/src/main/java/org/apache/hadoop/hbase/io/ByteBufferPool.java
+++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/io/ByteBufferPool.java
@@ -80,9 +80,9 @@ public class ByteBufferPool {
     this.maxPoolSize = maxPoolSize;
     this.directByteBuffer = directByteBuffer;
     // TODO can add initialPoolSize config also and make those many BBs ready for use.
-    LOG.info("Created with bufferSize={} and maxPoolSize={}",
-        org.apache.hadoop.util.StringUtils.byteDesc(bufferSize),
-        org.apache.hadoop.util.StringUtils.byteDesc(maxPoolSize));
+    LOG.info("Created with bufferSize={}, maxPoolSize={} and totalBufferSize={}",
+      org.apache.hadoop.util.StringUtils.byteDesc(bufferSize), maxPoolSize,
+      org.apache.hadoop.util.StringUtils.byteDesc((long) (bufferSize * maxPoolSize)));
     this.count = new AtomicInteger(0);
   }