----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/930/ -----------------------------------------------------------
Review request for hbase and Jonathan Gray. Summary ------- Fix is two-fold. First, added new facility where on successful open, we go and update the timers on all regions in transition that were on the same server. Secondly, in the timeout monitor, we'll do necessary cleanup and state transitions so that when we go into re-assign, we have the proper state M src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java Changed regionPlans to be a concurrentskiplist. Makes life easier and in no place do we need lock on regionPlans to span other than regionPlans changes. Added to the processing of successful region open, the cleanup of its regionPlan and a run of updateTimers. Put setOffline in place of some code that duplicated what it did. This addresses bug hbase-3068. http://issues.apache.org/jira/browse/hbase-3068 Diffs ----- trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 1003330 Diff: http://review.cloudera.org/r/930/diff Testing ------- Basic unit tests seem to be passing. Testing now up on cluster. Thanks, stack