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

HBase Review Board commented on HBASE-2694:
-------------------------------------------

Message from: "Karthik Ranganathan" <[email protected]>

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




> Move RS to Master region open/close messaging into ZooKeeper
> ------------------------------------------------------------
>
>                 Key: HBASE-2694
>                 URL: https://issues.apache.org/jira/browse/HBASE-2694
>             Project: HBase
>          Issue Type: Sub-task
>          Components: master, regionserver
>            Reporter: Jonathan Gray
>            Priority: Critical
>             Fix For: 0.21.0
>
>         Attachments: HBASE-2694-OPENSOURCE-TRUNK-zk-based-messaging-v2.patch, 
> HBASE-2694-OPENSOURCE-TRUNK-zk-based-messaging.patch
>
>
> As a first step towards HBASE-2485, this issue is about changing the message 
> flow of opening and closing of regions without actually changing the 
> implementation of what happens on both the Master and RegionServer sides.  
> This way we can debug the messaging changes before the introduction of more 
> significant changes to the master architecture and handling of regions in 
> transition.

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

Reply via email to