hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Gray" <jg...@apache.org>
Subject Re: Review Request: Need to retain DeadServers to ensure we don't allow previously expired RS instances to rejoin cluster
Date Mon, 29 Nov 2010 19:43:07 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-11-29 11:43:07.682958)

Review request for hbase and stack.


Makes DeadServers private.  It was still accessed by my TestRollingRestart test so I had to
make a small change to how that worked.

Also added some additional testing in TestDeadServers that verifies the new boolean check
works as expected and also that the max capacity works as expected.


We currently let go of dead servers once we finish their shutdown.  We should hang on to them
longer to deal with things like network partitions.

I'm not a fan of SoftReferences so I decided on another approach.  DeadServers now has a maximum
number of servers to hold on to in the set (default 100).  Once it reaches the max, it evicts
the oldest.

More code than I had hoped but nothing too crazy.

This addresses bug HBASE-3282.

Diffs (updated)

  branches/0.90/src/main/java/org/apache/hadoop/hbase/master/DeadServer.java 1040242 
  branches/0.90/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1040242 
  branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 1040242 
  branches/0.90/src/test/java/org/apache/hadoop/hbase/master/TestDeadServer.java 1040242 
  branches/0.90/src/test/java/org/apache/hadoop/hbase/master/TestRollingRestart.java 1040242

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


Running unit tests now.



View raw message