[HBASE-19114] Split out o.a.h.h.zookeeper from hbase-server and hbase-client

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0-beta-1, 2.0.0

Release Note

Splits out most of ZooKeeper related code into a separate new module: hbase-zookeeper.
Also, renames some ZooKeeper related classes to follow a common naming pattern - "ZK" prefix - as compared to many different styles earlier.

• Assignee: Apekshit Sharma
• Reporter: Apekshit Sharma
• Created: 27/Oct/17 23:45
• Updated: 21/Mar/18 22:20
• Resolved: 03/Mar/18 15:37

Description

Changes so far:

• Moved DrainingServerTracker and RegionServerTracker to hbase-server:o.a.h.h.master.
• Moved SplitOrMergeTracker to oahh.master (because it depends on a PB)
• Moving hbase-client:oahh.zookeeper.* to hbase-zookeeper module. After Apache9's cleanup (HBASE-19200), hbase-client doesn't need them anymore (except 3 classes).
• Renamed some classes to use a consistent naming for classes - ZK instead of mix of ZK, Zk , ZooKeeper. Couldn't rename following public classes: MiniZooKeeperCluster, ZooKeeperConnectionException. Left RecoverableZooKeeper for lack of better name. (suggestions?)
• Sadly, can't move tests out because they depend on HBaseTestingUtility (which defeats part of the purpose - trimming down hbase-server tests. We need to promote more use of mocks in our tests)

Issue Links

• is blocked by: HBASE-19200 make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
• links to: Old Review Board
• links to: Review Board (master)