You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Enis Soztutar (JIRA)" <ji...@apache.org> on 2012/11/02 19:35:12 UTC

[jira] [Updated] (HBASE-7063) Package name for compression should not contain hfile

     [ https://issues.apache.org/jira/browse/HBASE-7063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Enis Soztutar updated HBASE-7063:
---------------------------------

    Attachment: hbase-7063_v1.patch

Attaching pretty straightforward patch. 
{code}
renamed:    hbase-common/src/main/java/org/apache/hadoop/hbase/io/hfile/Compression.java -> hbase-common/src/main/java/org/apache/hadoop/hbase/io/compress/Compression.java
renamed:    hbase-common/src/main/java/org/apache/hadoop/hbase/io/hfile/ReusableStreamGzipCodec.java -> hbase-common/src/main/java/org/apache/hadoop/hbase/io/compress/ReusableStreamGzipCodec.java
{code}
The only change is, I've made Compression.getSupportedAlgorithms() public, instead of package-private. 
                
> Package name for compression should not contain hfile 
> ------------------------------------------------------
>
>                 Key: HBASE-7063
>                 URL: https://issues.apache.org/jira/browse/HBASE-7063
>             Project: HBase
>          Issue Type: Improvement
>          Components: HFile, io
>    Affects Versions: 0.96.0
>            Reporter: Enis Soztutar
>              Labels: noob
>         Attachments: hbase-7063_v1.patch
>
>
> Compression codecs does not have any hfile specific functionality, and can be used elsewhere (RPC, hlog, see: HBASE-6966)
> We should move o.a.h.h.io.hfile.Compession and related files from io.hfile pacakge. We can use io.compress to be in line with hadoop. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira