hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Gray" <jg...@apache.org>
Subject Review Request: HBASE-3047: If new master crashes, restart is messy
Date Tue, 28 Sep 2010 22:52:46 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for hbase, stack and Jonathan Gray.


This is patch from Stack, just putting up on rb.

M src/test/java/org/apache/hadoop/hbase/catalog/TestCatalogTracker.java
  Add test of case where HRegionInterface connection throws a
  ConnectionException. Also tests two new verify root and meta 
  locations added to CatalogTracker.
M src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
  Change order in which we start up trackers in ZK.  Also add blocking
  until master is up to make it less likely we'll start before master
  comes up, especially around the cluster start up situation.
M src/main/java/org/apache/hadoop/hbase/master/HMaster.java
  Introduce new state on startup, the case where the cluster is
  NOT a fresh startup and its NOT a cluster where all is fully
  assigned.  The repair the master needs run to fixup this new
  state is not yet done; we throw a NotImplementedException for
  now.  TODO.  Added new isRunningCluster checker used figuring
  what the cluster condition is when master is joining.  Not
  comprehensive but good enough for now.
M src/main/java/org/apache/hadoop/hbase/catalog/CatalogTracker.java
  Added new verifyRootRegionLocation and verifyMetaRegionLocation.
  Needed to verify whats in zk is actually locations of catalog
M src/main/java/org/apache/hadoop/hbase/ipc/HRegionInterface.java
  Add fact that the verifying method, getRegionInfo, can throw

This addresses bug HBASE-3047.


  trunk/src/main/java/org/apache/hadoop/hbase/catalog/CatalogTracker.java 1002359 
  trunk/src/main/java/org/apache/hadoop/hbase/ipc/HRegionInterface.java 1002359 
  trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1002359 
  trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 1002359 
  trunk/src/test/java/org/apache/hadoop/hbase/catalog/TestCatalogTracker.java 1002359 

Diff: http://review.cloudera.org/r/915/diff




View raw message