hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jean-Daniel Cryans" <jdcry...@apache.org>
Subject Review Request: LeaseStillHeldException totally ignored by RS, wrongly named
Date Tue, 08 Jun 2010 23:04:06 GMT

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

Review request for hbase.


Summary
-------

Patch that first adds a new testing facility to send exceptions to region servers when they
report in. 
Then I added YouAreDeadException and PleaseHoldException that replaces the ambiguous lease
still held exception, and both are treated differently.
Finally, I added a very simple test that makes sure that telling a RS that its dead will actually
end up in killing it.


This addresses bug HBASE-2691.


Diffs
-----

  /trunk/src/main/java/org/apache/hadoop/hbase/PleaseHoldException.java PRE-CREATION 
  /trunk/src/main/java/org/apache/hadoop/hbase/YouAreDeadException.java PRE-CREATION 
  /trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 952836 
  /trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 952836 
  /trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 952836 
  /trunk/src/test/java/org/apache/hadoop/hbase/MiniHBaseCluster.java 952836 
  /trunk/src/test/java/org/apache/hadoop/hbase/TestYouAreDead.java PRE-CREATION 

Diff: http://review.hbase.org/r/164/diff


Testing
-------


Thanks,

Jean-Daniel


Mime
View raw message