hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Gray (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-2696) ZooKeeper cleanup and refactor
Date Thu, 15 Jul 2010 17:15:49 GMT

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

Jonathan Gray commented on HBASE-2696:
--------------------------------------

Next set of changes for ZK cleanup (following master startup cleanup in HBASE-2695).

This completes move from ZooKeeperWrapper to ZooKeeperWatcher (and all the new ZK bits) for
the RS and Client.  There is still work to do in Master, that is next patch which hopefully
comes today.

* Rename {{ServerStatus}} to {{ServerController}} (still have MasterStatus, that will be renamed
in next patch)
** Client uses this now, maybe should be ProcessController?
* Adds a new client exception {{ZooKeeperConnectionException}} which is subclass of IOException.
 Thrown from HTable and HBaseAdmin if no ZK connection
* Fixes client so that it can ride over master changes
** Uses {{MasterAddressManager}} already being used on RS
** {{HBaseAdmin}} uses accessor method to get master reference rather than a class variable
** Move all ZK interactions to go via ZKUtil/ZKWatcher not ZKWrapper
* Moves RS completely off of ZKWrapper and onto ZKUtil/ZKWatcher
* Adds lots of new methods to ZKUtil, every one with full javadoc

> ZooKeeper cleanup and refactor
> ------------------------------
>
>                 Key: HBASE-2696
>                 URL: https://issues.apache.org/jira/browse/HBASE-2696
>             Project: HBase
>          Issue Type: Sub-task
>          Components: master, regionserver, zookeeper
>            Reporter: Jonathan Gray
>            Assignee: Jonathan Gray
>            Priority: Critical
>             Fix For: 0.90.0
>
>         Attachments: HBASE-2696-part1-NewClasses_NotIntegrated.patch, HBASE-2696-part1-v2-NewClasses_NotIntegrated.patch,
HBASE-2696-part1-v3-NewClasses_RS.patch, HBASE-2696-part1-v4-NewClasses_RS_Tested.patch, HBASE-2696-part1-v5-NewClasses_RS_Tested.patch
>
>
> Currently almost everything we do with ZooKeeper is stuffed into a single class {{ZookeeperWrapper}}.
> This issue will deal with cleaning up our usage of ZK, adding some new abstractions to
help with the master changes, splitting up watchers from utility methods, and nailing down
the contracts of our ZK methods with respect to setting watchers, throwing exceptions, etc...

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message