[
https://issues.apache.org/jira/browse/HBASE-2988?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12909105#action_12909105
]
HBase Review Board commented on HBASE-2988:
-------------------------------------------
Message from: "Andrew Purtell" <[email protected]>
bq. On 2010-09-13 10:16:21, Todd Lipcon wrote:
bq. > src/main/java/org/apache/hadoop/hbase/HColumnDescriptor.java, line 368
bq. > <http://review.cloudera.org/r/821/diff/1/?file=11487#file11487line368>
bq. >
bq. > This will throw NPE if it's not set, right?
bq. > Why not just make this return getCompression() if n == null, then
you can get rid of the hasCompactionCompression() check down in StoreFile below
and simplify things a bit?
bq. >
bq. > Also why redundant getCompactionCompression and
getCompactionCompressionType?
There is also getCompaction and getCompactionType.
- Andrew
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/821/#review1163
-----------------------------------------------------------
> Support alternate compression for major compactions
> ---------------------------------------------------
>
> Key: HBASE-2988
> URL: https://issues.apache.org/jira/browse/HBASE-2988
> Project: HBase
> Issue Type: Improvement
> Reporter: Andrew Purtell
> Assignee: Andrew Purtell
> Priority: Minor
> Attachments: HBASE-2988-0.20.patch, HBASE-2988-trunk.patch
>
>
> Support an alternate compression scheme on HFiles during major compaction.
> See discussion on HBASE-2987 for background.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
