[ 
https://issues.apache.org/jira/browse/HBASE-20625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16510565#comment-16510565
 ] 

Jingyun Tian commented on HBASE-20625:
--------------------------------------

[~zghaobac] do you mean remove setComprssionContext at this patch? or 3.0? 

> refactor some WALCellCodec related code
> ---------------------------------------
>
>                 Key: HBASE-20625
>                 URL: https://issues.apache.org/jira/browse/HBASE-20625
>             Project: HBase
>          Issue Type: Improvement
>          Components: wal
>    Affects Versions: 2.0.0
>            Reporter: Jingyun Tian
>            Assignee: Jingyun Tian
>            Priority: Minor
>             Fix For: 3.0.0
>
>         Attachments: HBASE-20625.master.001.patch, 
> HBASE-20625.master.002.patch, HBASE-20625.master.002.patch, 
> HBASE-20625.master.003.patch, HBASE-20625.master.004.patch, 
> HBASE-20625.master.005.patch, HBASE-20625.master.006.patch
>
>
> Currently I'm working on export HLog to another FileSystem, then I found the 
> code of WALCellCodec and  its related classes is not that clean. And there 
> are several TODOs. Thus I tried to refactor the code based one these TODOs. 
> e.g.
> {code}
>   // TODO: it sucks that compression context is in WAL.Entry. It'd be nice if 
> it was here.
>   //       Dictionary could be gotten by enum; initially, based on enum, 
> context would create
>   //       an array of dictionaries.
>   static class BaosAndCompressor extends ByteArrayOutputStream implements 
> ByteStringCompressor {
>     public ByteString toByteString() {
>       // We need this copy to create the ByteString as the byte[] 'buf' is 
> not immutable. We reuse
>       // them.
>       return ByteString.copyFrom(this.buf, 0, this.count);
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to