----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/336/#review445 -----------------------------------------------------------
trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java <http://review.hbase.org/r/336/#comment1872> you should use initHRegion in this class instead of repeating the boilerplate. trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java <http://review.hbase.org/r/336/#comment1871> this might not be necessary, since flushcache() returns after the file has been loaded into HRegion/Stores. - Ryan On 2010-07-20 11:19:26, Pranav Khaitan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.hbase.org/r/336/ > ----------------------------------------------------------- > > (Updated 2010-07-20 11:19:26) > > > Review request for hbase, Nicolas, Karthik Ranganathan, and Kannan > Muthukkaruppan. > > > Summary > ------- > > Fixed bloom filter code in StoreFile > > > This addresses bug HBASE-2852. > http://issues.apache.org/jira/browse/HBASE-2852 > > > Diffs > ----- > > trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java > 965615 > trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java > 965615 > > Diff: http://review.hbase.org/r/336/diff > > > Testing > ------- > > JUnit tests added > > > Thanks, > > Pranav > >
