hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Gray" <jg...@apache.org>
Subject Re: Review Request: Add TimeRange support into Increment to optimize for counters that are partitioned on time
Date Sat, 30 Oct 2010 04:53:51 GMT


> On 2010-10-29 21:19:11, Ryan Rawson wrote:
> > trunk/src/main/java/org/apache/hadoop/hbase/client/Increment.java, line 274
> > <http://review.cloudera.org/r/1132/diff/1/?file=16260#file16260line274>
> >
> >     technically Increment.java has never shipped in a published version of HBase
so we wont need to bump any version numbers I think.

+1


- Jonathan


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


On 2010-10-29 19:42:21, Jonathan Gray wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.cloudera.org/r/1132/
> -----------------------------------------------------------
> 
> (Updated 2010-10-29 19:42:21)
> 
> 
> Review request for hbase, stack, Ryan Rawson, and khemani.
> 
> 
> Summary
> -------
> 
> TimeRange now part of Increment.  Adds new methods into Increment w/ javadoc.  Adds code
in our increment operation to set this onto the Gets we use.
> 
> 
> This addresses bug HBASE-3162.
>     http://issues.apache.org/jira/browse/HBASE-3162
> 
> 
> Diffs
> -----
> 
>   trunk/src/main/java/org/apache/hadoop/hbase/client/Increment.java 1028936 
>   trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 1028936 
> 
> Diff: http://review.cloudera.org/r/1132/diff
> 
> 
> Testing
> -------
> 
> increment tests passing.
> 
> 
> Thanks,
> 
> Jonathan
> 
>


Mime
View raw message