[
https://issues.apache.org/jira/browse/HBASE-20625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16509332#comment-16509332
]
Jingyun Tian commented on HBASE-20625:
--------------------------------------
[~zghaobac] thx for review.
{quote}
The new impl is Short.MAX_VALUE? Why change this?
{quote}
I think they are all Short.MAX_VALUE... fixed..
The other 2 problems are also fixed.
The only use of setCompressionContext now is in WAL, and I deprecated it, too.
There are no other uses to these two method.
> 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)
