[
https://issues.apache.org/jira/browse/HBASE-10866?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13951816#comment-13951816
]
Mikhail Antonov commented on HBASE-10866:
-----------------------------------------
[~stack] - thanks for review!
Yeah, full decoupling of log splitting from zookeeper will involve also changes
in zookeeper listener on RS side (SplitLogWorker), and changes in HMaster
(SplitLogManager). The former one is what I'm working on, and master side I
will tackle after that.
Regarding packaging - definitely both master and RS would include logic for
consensus, and as more patches in this direction are rolled out, we should see
that. In that patch I'm trying to keep it narrow in scope, so that it's easier
for initial review and discuss.
So I think I'll write up a short document and post in for discussion sometime
early-mid next week (unless you suggest otherwise).
W.r.t interface conventions- I'll fix them and revise the patch over weekend,
will also check the tests which failed.
> Decouple HLogSplitterHandler from ZooKeeper
> -------------------------------------------
>
> Key: HBASE-10866
> URL: https://issues.apache.org/jira/browse/HBASE-10866
> Project: HBase
> Issue Type: Improvement
> Components: regionserver, Zookeeper
> Reporter: Mikhail Antonov
> Attachments: HBASE-10866.patch
>
>
> As some sort of follow-up or initial step towards HBASE-10296...
> Whatever consensus algorithm/library may be the chosen, perhaps on of first
> practical steps towards this goal would be to better abstract ZK-related API
> and details, which are now throughout the codebase (mostly leaked throuth
> ZkUtil, ZooKeeperWatcher and listeners).
> I'd like to propose a series of patches to help better abstract out zookeeper
> (and then help develop consensus APIs).
> Here is first version of patch for initial review (then I'm planning to work
> on another handlers in regionserver, and then perhaps start working on
> abstracting listeners).
--
This message was sent by Atlassian JIRA
(v6.2#6252)
