-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/930/
-----------------------------------------------------------
(Updated 2010-10-01 14:05:45.726776)
Review request for hbase and Jonathan Gray.
Changes
-------
Update to javadoc and comments
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 (updated)
-----
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
