-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1187/
-----------------------------------------------------------
Review request for hbase and Jonathan Gray.
Summary
-------
Renamed ZKTableDisable as ZKTable, making it a generic zk util for managing
'tables'.
Added enabing/disabling states to table the current set of enabled/disabled
only.
M src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
(createSetData): Added.
M src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
Removed offlining region utility methods no longer used.
(We do it now over in MetaEditor)
M src/main/java/org/apache/hadoop/hbase/HRegionInfo.java
Javadoc.
M src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
Add a base abstract class to do 'bulk assignments'. Redo
assignAllUserRegions to use subclass of new bulk assigner class.
Added isTableEnabled, disablingTable, enablingTable.
M src/main/java/org/apache/hadoop/hbase/master/handler/DisableTableHandler.java
Redid to use new bulk assigner class.
M src/main/java/org/apache/hadoop/hbase/master/handler/EnableTableHandler.java
A Added TestZKTable
This addresses bug hbase-3112.
http://issues.apache.org/jira/browse/hbase-3112
Diffs
-----
trunk/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java 1032652
trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
1032652
trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
1032652
trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1032652
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/DeleteTableHandler.java
1032652
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/DisableTableHandler.java
1032652
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/EnableTableHandler.java
1032652
trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 1032652
trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKTableDisable.java
1032652
trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java 1032652
trunk/src/test/java/org/apache/hadoop/hbase/master/TestMasterFailover.java
1032652
trunk/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZKTable.java
PRE-CREATION
Diff: http://review.cloudera.org/r/1187/diff
Testing
-------
Thanks,
stack