[
https://issues.apache.org/jira/browse/HBASE-10240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16151551#comment-16151551
]
Chia-Ping Tsai edited comment on HBASE-10240 at 9/2/17 5:34 PM:
----------------------------------------------------------------
bq. Can we purge any of the remaining Writables?
There are only three remaining Writables - TablePermission, TimeRangeTracker,
and MetaWriter.
For TablePermission andTimeRangeTracker, we can do something like this:
# add the wire-compatible function for reading (we had already done this for
TablePermission. ditto for TimeRangeTracker.)
# rewrite the writable data as PB encoded when reading
# replace the Writable with inner delegation. Or make the write/readFields
static. (purge public Writable inheritance)
For MetaWriter, we just introduce another interface to replace Writable. The
readFields is useless.
was (Author: chia7712):
bq. Can we purge any of the remaining Writables?
There are only three remaining Writables - TablePermission, TimeRangeTracker,
and MetaWriter.
For TablePermission andTimeRangeTracker, we can do something like this:
# add the wire-compatible function for reading (we had already done this for
TablePermission. ditto for TimeRangeTracker.)
# rewrite the writable data as PB encoded when reading
# replace the Writable with inner delegation. Or make the write/readFields
static. (purge public Writable inheritance)
For MetaWriter, we just introduce another interface to replace Writable. The
readFields is useless.
> Remove 0.94->0.96 migration code
> --------------------------------
>
> Key: HBASE-10240
> URL: https://issues.apache.org/jira/browse/HBASE-10240
> Project: HBase
> Issue Type: Improvement
> Reporter: Andrew Purtell
> Assignee: Peter Somogyi
> Priority: Critical
> Fix For: 2.0.0
>
> Attachments: HBASE-10240.master.001.patch,
> HBASE-10240.master.001.patch
>
>
> Remove the objects and code only needed for supporting migration to 0.96 from
> 0.94.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
