You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by st...@duboce.net on 2010/09/16 19:08:37 UTC

Re: Review Request: Add reseek optimization for fetching multiple versions via TimestampsFilter

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/859/#review1244
-----------------------------------------------------------

Ship it!


+1

Ryan, you want to take a look since you've got your head in here?

- stack


On 2010-09-16 01:47:47, Pranav Khaitan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.cloudera.org/r/859/
> -----------------------------------------------------------
> 
> (Updated 2010-09-16 01:47:47)
> 
> 
> Review request for hbase and Kannan Muthukkaruppan.
> 
> 
> Summary
> -------
> 
> Skip between timestamps to avoid the unnecessary read for all the blocks which come in between.
> 
> For example:
> 
> If we have timestamps from 1 to 1000 for a particular row, family, qualifier and we want to read only timestamps 990 and 10, then we should be able to read 990 and then jump to 10 instead of doing a sequential read including everything in between.
> 
> 
> This addresses bug HBASE-3005.
>     http://issues.apache.org/jira/browse/HBASE-3005
> 
> 
> Diffs
> -----
> 
>   trunk/src/main/java/org/apache/hadoop/hbase/KeyValue.java 997639 
>   trunk/src/main/java/org/apache/hadoop/hbase/filter/TimestampsFilter.java 997639 
> 
> Diff: http://review.cloudera.org/r/859/diff
> 
> 
> Testing
> -------
> 
> All tests using TimestampFilter passing
> 
> Reviewed by Kannan
> 
> 
> Thanks,
> 
> Pranav
> 
>