[
https://issues.apache.org/jira/browse/HBASE-20625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16509645#comment-16509645
]
Guanghao Zhang commented on HBASE-20625:
----------------------------------------
[~stack] Can we remove a public method from a IA.LimitPrivate interface when
take a major release, etc. 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)
