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

Review request for hbase, stack and khemani.


Summary
-------

Adds a new Increment class that allows multiple columns (each w/ own increment 
amount) in a single row being incremented in one call.

The big wins here are being able to do multiple columns in a row in a single 
RPC and having it be appended/synced to the WAL in a single call.

The current trade-off is that you lose atomicity to readers (ie. this does not 
currently use RWCC).  Eventually it could but for the current use case I am 
building this for, it's okay like this.


This addresses bug HBASE-2946.
    http://issues.apache.org/jira/browse/HBASE-2946


Diffs
-----

  trunk/src/main/java/org/apache/hadoop/hbase/client/HTable.java 1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/client/HTableInterface.java 
1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/client/Increment.java 
PRE-CREATION 
  trunk/src/main/java/org/apache/hadoop/hbase/io/HbaseObjectWritable.java 
1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/ipc/HRegionInterface.java 1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 
1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/regionserver/MemStore.java 
1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java 1026930 
  trunk/src/main/java/org/apache/hadoop/hbase/rest/client/RemoteHTable.java 
1026930 
  trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java 
1026930 

Diff: http://review.cloudera.org/r/1088/diff


Testing
-------

Added TestFromClientSide.testIncrement() which adds some client-side tests of 
Increment (and mixing w/ original icv call).  That passes and most the way 
through a test suite run.


Thanks,

Jonathan

Reply via email to