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: Fix periodic major compaction (HBASE-2990) and expiration check (HBASE-3083)
Date Mon, 01 Nov 2010 23:31:36 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-11-01 16:31:36.124095)

Review request for hbase, stack and khemani.


Latest version.  I guess I should write some tests...


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.

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




View raw message