----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/171/ -----------------------------------------------------------
Review request for hbase, Todd Lipcon and stack. Summary ------- Moved all RS -> Master communication to go through ZK so that we can enable master failover. This addresses bug HBASE-2694. http://issues.apache.org/jira/browse/HBASE-2694 Diffs ----- trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/executor/HBaseEventHandler.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/executor/HBaseExecutorService.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/executor/NamedThreadFactory.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/executor/RegionTransitionEventData.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/ProcessRegionClose.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/ProcessRegionStatusChange.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/RegionManager.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/ZKMasterAddressWatcher.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/master/ZKUnassignedWatcher.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/master/handler/MasterCloseRegionHandler.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/master/handler/MasterOpenRegionHandler.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RSZookeeperUpdater.java PRE-CREATION trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/HQuorumPeer.java 953804 trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWrapper.java 953804 trunk/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java 953804 trunk/src/test/java/org/apache/hadoop/hbase/TestZooKeeper.java 953804 trunk/src/test/java/org/apache/hadoop/hbase/master/TestMaster.java 953804 trunk/src/test/java/org/apache/hadoop/hbase/master/TestZKBasedCloseRegion.java PRE-CREATION trunk/src/test/java/org/apache/hadoop/hbase/master/TestZKBasedReopenRegion.java PRE-CREATION Diff: http://review.hbase.org/r/171/diff Testing ------- Unit tested. Thanks, Karthik