----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1089/ -----------------------------------------------------------
(Updated 2010-11-01 16:31:36.124095) Review request for hbase, stack and khemani. Changes ------- Latest version. I guess I should write some tests... Summary ------- This is a somewhat misguided attempt. It's not done but it shows the fairly simple change to the actual major compaction code. The hard part is: + long lastMajor = region.getRegionInfo().getRegionData().getLastMajor(); And the question is where to persist that. This patch adds a new class called HRegionData into HRegionInfo that contains any number of key-value pairs of data that get persisted with the HRI. Not really sure how I ended up there but this data seemed like an odd-man-out so adding another field seemed weird. We also need some kind of versioning in HRI so we can add stuff w/o migrating. There's some versioned stuff in HRData. Just looking for some feedback / ideas. This addresses bugs HBASE-2990 and HBASE-3083. http://issues.apache.org/jira/browse/HBASE-2990 http://issues.apache.org/jira/browse/HBASE-3083 Diffs (updated) ----- trunk/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java 1029789 trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1029789 trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionData.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 1029789 trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java 1029789 Diff: http://review.cloudera.org/r/1089/diff Testing ------- Thanks, Jonathan