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

Review request for hbase and stack.


Summary
-------

Adds new handling of the timeouts for PENDING_OPEN and PENDING_CLOSE in-memory 
master RIT states.

Adds some new broken RIT states into TestMasterFailover.

Some of these broken states don't seem possible to me but as long as we aren't 
breaking the existing behaviors and tests I think it's okay if we handle odd 
cases that can be mocked.  Who knows what will happen in the real world.

The reason TestMasterFailover didn't/doesn't really test for the issue in 
HBASE-3147 is this new broken condition happens when an RS dies / goes offline 
rather than a master failover concurrent w/ RS failure.


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


Diffs
-----

  trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 
1026911 
  trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 1026911 
  trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKAssign.java 1026911 
  trunk/src/test/java/org/apache/hadoop/hbase/master/TestMasterFailover.java 
1026911 

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


Testing
-------

TestMasterFailover passes.


Thanks,

Jonathan

Reply via email to