----------------------------------------------------------- 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 > >
