----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1053/#review1596 -----------------------------------------------------------
src/main/java/org/apache/hadoop/hbase/regionserver/InternalScan.java <http://review.cloudera.org/r/1053/#comment5396> Why not put these into the base class? Why create an entirely new class just to hold these? src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java <http://review.cloudera.org/r/1053/#comment5397> Would this code be cleaner if we didnt have to use instanceof? - Ryan On 2010-10-20 21:31:17, khemani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.cloudera.org/r/1053/ > ----------------------------------------------------------- > > (Updated 2010-10-20 21:31:17) > > > Review request for hbase and Jonathan Gray. > > > Summary > ------- > > Ensure that during incrementColumnValue() the scan triggered by the get() > does an early exit if it finds the KV in the memstore. > > > This addresses bug HBASE-3082. > http://issues.apache.org/jira/browse/HBASE-3082 > > > Diffs > ----- > > src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 0e3940c > src/main/java/org/apache/hadoop/hbase/regionserver/InternalScan.java > PRE-CREATION > src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java > 4775fc8 > > Diff: http://review.cloudera.org/r/1053/diff > > > Testing > ------- > > I have been testing it on my cluster. No unit testing yet. > > > Thanks, > > khemani > >
