[ 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.