hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Gray" <jg...@apache.org>
Subject Review Request: Add separate handling of PENDING_OPEN/PENDING_CLOSE in timeout monitor and additional testing
Date Mon, 25 Oct 2010 01:12:10 GMT

-----------------------------------------------------------
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


Mime
View raw message