-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/915/
-----------------------------------------------------------
(Updated 2010-09-29 10:23:49.845051)
Review request for hbase, stack and Jonathan Gray.
Changes
-------
More cleanup
Summary
-------
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
Javadoc.
Added new verifyRootRegionLocation and verifyMetaRegionLocation.
Needed to verify whats in zk is actually locations of catalog
regions.
M src/main/java/org/apache/hadoop/hbase/ipc/HRegionInterface.java
Add fact that the verifying method, getRegionInfo, can throw
ConnectException
This addresses bug HBASE-3047.
http://issues.apache.org/jira/browse/HBASE-3047
Diffs (updated)
-----
trunk/src/main/java/org/apache/hadoop/hbase/RemoteExceptionHandler.java
1001981
trunk/src/main/java/org/apache/hadoop/hbase/catalog/CatalogTracker.java
1001981
trunk/src/main/java/org/apache/hadoop/hbase/ipc/HRegionInterface.java 1001981
trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
1001981
trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1001981
trunk/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java
1001981
trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 1001981
trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
1001981
trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Leases.java 1001981
trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperNodeTracker.java
1001981
trunk/src/main/resources/hbase-default.xml 1001981
trunk/src/test/java/org/apache/hadoop/hbase/catalog/TestCatalogTracker.java
1001981
Diff: http://review.cloudera.org/r/915/diff
Testing
-------
Thanks,
Jonathan
