Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-14421 TestFastFail* are flakey; ADDENDUM 3 Exclude another should assert in TestFastFail

This commit is contained in:
stack 2015-09-14 08:32:52 -07:00
parent 99df022f2c
commit 63a2fb5143
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFastFail.java
View File

@ -268,13 +268,12 @@ public class TestFastFail {
assertEquals("The regionservers that returned true should equal to the" assertEquals("The regionservers that returned true should equal to the"
+ " number of successful threads", numThreadsReturnedTrue, + " number of successful threads", numThreadsReturnedTrue,
numSuccessfullThreads.get()); numSuccessfullThreads.get());
assertTrue(
"There should be atleast one thread that retried instead of failing",
MyPreemptiveFastFailInterceptor.numBraveSouls.get() > 0);
/* 'should' is not worthy of an assert. Disabling because randomly this seems to randomly /* 'should' is not worthy of an assert. Disabling because randomly this seems to randomly
* not but true. St.Ack 20151012 * not but true. St.Ack 20151012
* *
assertTrue(
"There should be atleast one thread that retried instead of failing",
MyPreemptiveFastFailInterceptor.numBraveSouls.get() > 0);
assertTrue( assertTrue(
"There should be atleast one PreemptiveFastFail exception," "There should be atleast one PreemptiveFastFail exception,"
+ " otherwise, the test makes little sense." + " otherwise, the test makes little sense."