You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by Kris Jirapinyo <kj...@attensity.com> on 2010/07/12 07:54:42 UTC
Review Request: HBASE-2794 Multi Column Bloom Filter Check
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/296/
-----------------------------------------------------------
Review request for hbase.
Summary
-------
HBASE-2794 Enable bloom filter checks for multiple columns in same column family
This addresses bug HBASE-2794.
http://issues.apache.org/jira/browse/HBASE-2794
Diffs
-----
/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java 962748
/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestStoreFile.java 962748
Diff: http://review.hbase.org/r/296/diff
Testing
-------
Ran and passed org.apache.hadoop.hbase.regionserver.TestStoreFile multiple times. Ran and passed all tests when building.
Thanks,
Kris
Re: Review Request: HBASE-2794 Multi Column Bloom Filter Check
Posted by Kris Jirapinyo <kj...@attensity.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/296/
-----------------------------------------------------------
(Updated 2010-07-13 16:32:18.729301)
Review request for hbase.
Changes
-------
Added changes to code after HBASE-2265 was committed.
Also, incorporated suggestion from Nicolas to not lookup when columns.size*error.rate > 10%.
Changed BloomFilter interface, adding getErrorRate(). ByteBloomFilter now also has errorRate stored.
Summary
-------
HBASE-2794 Enable bloom filter checks for multiple columns in same column family
This addresses bug HBASE-2794.
http://issues.apache.org/jira/browse/HBASE-2794
Diffs (updated)
-----
/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java 963862
/trunk/src/main/java/org/apache/hadoop/hbase/util/BloomFilter.java 963873
/trunk/src/main/java/org/apache/hadoop/hbase/util/ByteBloomFilter.java 963873
/trunk/src/main/java/org/apache/hadoop/hbase/util/DynamicByteBloomFilter.java 963873
/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestStoreFile.java 963873
Diff: http://review.hbase.org/r/296/diff
Testing
-------
Ran and passed org.apache.hadoop.hbase.regionserver.TestStoreFile multiple times. Ran and passed all tests when building.
Thanks,
Kris
Re: Review Request: HBASE-2794 Multi Column Bloom Filter Check
Posted by Kris Jirapinyo <kj...@attensity.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/296/
-----------------------------------------------------------
(Updated 2010-07-12 19:48:43.373418)
Review request for hbase.
Changes
-------
Implemented Kannan's suggestion, thereby removing keyList.
Summary
-------
HBASE-2794 Enable bloom filter checks for multiple columns in same column family
This addresses bug HBASE-2794.
http://issues.apache.org/jira/browse/HBASE-2794
Diffs (updated)
-----
/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java 962748
/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestStoreFile.java 962748
Diff: http://review.hbase.org/r/296/diff
Testing
-------
Ran and passed org.apache.hadoop.hbase.regionserver.TestStoreFile multiple times. Ran and passed all tests when building.
Thanks,
Kris