hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Todd Lipcon" <t...@cloudera.com>
Subject Re: Review Request: Support alternate compression for major compactions
Date Tue, 14 Sep 2010 17:15:59 GMT

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



src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java
<http://review.cloudera.org/r/821/#comment4037>

    don't you want to return getCompression() in this type? ie compaction compression defaults
to "same as the rest of the table" unless specifically overridden?



src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
<http://review.cloudera.org/r/821/#comment4038>

    this way you can't specify "I don't want to compress majors"... probably unlikely, but
we should probably support it


- Todd


On 2010-09-13 20:20:49, Andrew Purtell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.cloudera.org/r/821/
> -----------------------------------------------------------
> 
> (Updated 2010-09-13 20:20:49)
> 
> 
> Review request for hbase.
> 
> 
> Summary
> -------
> 
> Support alternate compression for major compactions.
> 
> This is expected to be an uncommmon configuration so I did not pollute the HColumnDescriptor
constructor with the new option; instead only added convenience {get,set}ters.
> 
> 
> This addresses bug HBASE-2988.
>     http://issues.apache.org/jira/browse/HBASE-2988
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java 8e3bd53 
>   src/main/java/org/apache/hadoop/hbase/regionserver/Store.java 4f777f0 
>   src/main/ruby/hbase/admin.rb 82d7e54 
> 
> Diff: http://review.cloudera.org/r/821/diff
> 
> 
> Testing
> -------
> 
> Created table with LZO compression, inserted 10GB of data, altered schema with COMPRESSION_COMPACT
of 'LZMA' (custom LZMA Hadoop compression plugin), initiated major compaction, confirmed the
results by examining HFiles on local fs with the UNIX file utility.
> 
> 
> Thanks,
> 
> Andrew
> 
>


Mime
View raw message