-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1143/
-----------------------------------------------------------

Review request for hbase and stack.


Summary
-------

Does cleanup of RIT timeouts according to document in progress.  Still 
finishing document but I'd like to get this patch tested before finalizing it.

Also found some strange stuff in server shutdown handling that could have 
easily led to some double assignment issues that stack was seeing.


This addresses bug HBASE-3181.
    http://issues.apache.org/jira/browse/HBASE-3181


Diffs
-----

  trunk/src/main/java/org/apache/hadoop/hbase/HRegionInfo.java 1029466 
  trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 
1029466 
  trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1029466 
  
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java
 1029466 
  
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/EnableTableHandler.java
 1029466 
  
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java
 1029466 
  
trunk/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java
 1029466 
  trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKAssign.java 1029466 

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


Testing
-------

Working on tests now.  This definitely changes some behavior that is tested in 
the new TestMasterFailover so need to figure if the test should change or 
whether we need to handle things like CLOSING.  Maybe let it timeout a few 
times?


Thanks,

Jonathan

Reply via email to