liuml07 commented on a change in pull request #822: HBASE-23288 - Backport 
HBASE-23251 (Add Column Family and Table Names…
URL: https://github.com/apache/hbase/pull/822#discussion_r346196976
 
 

 ##########
 File path: 
hbase-common/src/main/java/org/apache/hadoop/hbase/io/hfile/HFileContext.java
 ##########
 @@ -203,11 +218,18 @@ public String getHFileName() {
   public long heapSize() {
     long size = ClassSize.align(ClassSize.OBJECT +
         // Algorithm reference, encodingon, checksumtype, Encryption.Context 
reference
-        5 * ClassSize.REFERENCE +
+        7 * ClassSize.REFERENCE +
 
 Review comment:
   Seems this is the only difference when cherry-picking? I'd rather using the 
same expression as HBASE-23251 if possible so future change would find 
comfortable eyeballs.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to