You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2011/08/04 03:20:50 UTC
svn commit: r1153718 - in /hbase/trunk: CHANGES.txt
src/test/java/org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java
Author: tedyu
Date: Thu Aug 4 01:20:49 2011
New Revision: 1153718
URL: http://svn.apache.org/viewvc?rev=1153718&view=rev
Log:
HBASE-3857 Fix TestHFileBlock.testBlockHeapSize test failure (Mikhail)
Modified:
hbase/trunk/CHANGES.txt
hbase/trunk/src/test/java/org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java
Modified: hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1153718&r1=1153717&r2=1153718&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Thu Aug 4 01:20:49 2011
@@ -184,6 +184,7 @@ Release 0.91.0 - Unreleased
HBASE-4032 HBASE-451 improperly breaks public API HRegionInfo#getTableDesc
HBASE-4148 HFileOutputFormat doesn't fill in TIMERANGE_KEY metadata (Jonathan Hsieh)
HBASE-4003 Cleanup Calls Conservatively On Timeout (Karthick)
+ HBASE-3857 Fix TestHFileBlock.testBlockHeapSize test failure (Mikhail)
IMPROVEMENTS
HBASE-3290 Max Compaction Size (Nicolas Spiegelberg via Stack)
Modified: hbase/trunk/src/test/java/org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/test/java/org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java?rev=1153718&r1=1153717&r2=1153718&view=diff
==============================================================================
--- hbase/trunk/src/test/java/org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java (original)
+++ hbase/trunk/src/test/java/org/apache/hadoop/hbase/io/hfile/TestHFileBlock.java Thu Aug 4 01:20:49 2011
@@ -482,12 +482,16 @@ public class TestHFileBlock {
@Test
public void testBlockHeapSize() {
+ // We have seen multiple possible values for this estimate of the heap size
+ // of a ByteBuffer, presumably depending on the JDK version.
+ assertTrue(HFileBlock.BYTE_BUFFER_HEAP_SIZE == 64 ||
+ HFileBlock.BYTE_BUFFER_HEAP_SIZE == 80);
+
for (int size : new int[] { 100, 256, 12345 }) {
byte[] byteArr = new byte[HFileBlock.HEADER_SIZE + size];
ByteBuffer buf = ByteBuffer.wrap(byteArr, 0, size);
HFileBlock block = new HFileBlock(BlockType.DATA, size, size, -1, buf,
true, -1);
- assertEquals(80, HFileBlock.BYTE_BUFFER_HEAP_SIZE);
long expected = ClassSize.align(ClassSize.estimateBase(HFileBlock.class,
true)
+ ClassSize.estimateBase(buf.getClass(), true)