> On 2010-06-08 16:18:09, stack wrote: > > /trunk/src/test/java/org/apache/hadoop/hbase/TestYouAreDead.java, line 27 > > <http://review.hbase.org/r/164/diff/1/?file=1103#file1103line27> > > > > These single test classes are killing us...regards overall unit tests > > times... can we not put it into an existing test class....?
Will be all merged into TestMasterWrongRS, that I rename to TestKillingServersFromMaster - Jean-Daniel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.hbase.org/r/164/#review160 ----------------------------------------------------------- On 2010-06-08 16:04:06, Jean-Daniel Cryans wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://review.hbase.org/r/164/ > ----------------------------------------------------------- > > (Updated 2010-06-08 16:04:06) > > > 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 > >
