Author: nspiegelberg
Date: Thu Jan  5 02:37:59 2012
New Revision: 1227452

URL: http://svn.apache.org/viewvc?rev=1227452&view=rev
Log:
[HBASE-5021] Add one LONG overhead to the fixed heap size of HRegion

Summary:
There is a new attribute introduced to HRegion class by the commit
(https://phabricator.fb.com/rH21326)

Tag it as private because it is minor problem.
Or we could use the same jira number ?

Test Plan: TestHeapSize

Reviewers: nspiegelberg, mbautin

Reviewed By: mbautin

CC: hbase-eng@lists, mbautin, nspiegelberg

Differential Revision: 381528

Modified:
    
hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Modified: 
hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
URL: 
http://svn.apache.org/viewvc/hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java?rev=1227452&r1=1227451&r2=1227452&view=diff
==============================================================================
--- 
hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
 (original)
+++ 
hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
 Thu Jan  5 02:37:59 2012
@@ -3591,7 +3591,7 @@ public class HRegion implements HeapSize
   }
 
   public static final long FIXED_OVERHEAD = ClassSize.align(
-      (4 * Bytes.SIZEOF_LONG) + 2 * ClassSize.ARRAY +
+      (5 * Bytes.SIZEOF_LONG) + 2 * ClassSize.ARRAY +
       (25 * ClassSize.REFERENCE) + ClassSize.OBJECT + Bytes.SIZEOF_INT);
 
   public static final long DEEP_OVERHEAD = ClassSize.align(FIXED_OVERHEAD +


Reply via email to