Author: nspiegelberg
Date: Thu Mar 17 19:26:51 2011
New Revision: 1082653

URL: http://svn.apache.org/viewvc?rev=1082653&view=rev
Log:
HBASE-3665 tighten assertions for testBloomFilterSize

Modified:
    
hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java

Modified: 
hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java?rev=1082653&r1=1082652&r2=1082653&view=diff
==============================================================================
--- 
hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java 
(original)
+++ 
hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java 
Thu Mar 17 19:26:51 2011
@@ -2794,11 +2794,12 @@ public class TestHRegion extends HBaseTe
     int duplicate_multiplier =2;
     int num_storefiles = 4;
 
+    int version = 0;
     for (int f =0 ; f < num_storefiles; f++) {
       for (int i = 0; i < duplicate_multiplier; i ++) {
         for (int j = 0; j < num_unique_rows; j++) {
           Put put = new Put(Bytes.toBytes("row" + j));
-          put.add(fam1, qf1, val1);
+                       put.add(fam1, qf1, version++, val1);
           region.put(put);
         }
       }
@@ -2811,6 +2812,7 @@ public class TestHRegion extends HBaseTe
       StoreFile.Reader reader = storefile.getReader();
       reader.loadFileInfo();
       reader.loadBloomfilter();
+      assertEquals(num_unique_rows*duplicate_multiplier, reader.getEntries());
       assertEquals(num_unique_rows, reader.getFilterEntries());
     }
     
@@ -2822,6 +2824,8 @@ public class TestHRegion extends HBaseTe
       StoreFile.Reader reader = storefile.getReader();
       reader.loadFileInfo();
       reader.loadBloomfilter();
+      assertEquals(num_unique_rows*duplicate_multiplier*num_storefiles,
+          reader.getEntries());
       assertEquals(num_unique_rows, reader.getFilterEntries());
     }  
   }


Reply via email to