HDFS-6631. TestPread#testHedgedReadLoopTooManyTimes fails intermittently. Contributed by Liang Xie.
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1609534 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
5279f64cb6
commit
cce02e3bef
|
@ -807,6 +807,9 @@ Release 2.5.0 - UNRELEASED
|
|||
|
||||
HDFS-5411. Update Bookkeeper dependency to 4.2.3. (Rakesh R via umamahesh)
|
||||
|
||||
HDFS-6631. TestPread#testHedgedReadLoopTooManyTimes fails intermittently.
|
||||
(Liang Xie via cnauroth)
|
||||
|
||||
BREAKDOWN OF HDFS-2006 SUBTASKS AND RELATED JIRAS
|
||||
|
||||
HDFS-6299. Protobuf for XAttr and client-side implementation. (Yi Liu via umamahesh)
|
||||
|
|
|
@ -52,4 +52,6 @@ public class DFSClientFaultInjector {
|
|||
public void startFetchFromDatanode() {}
|
||||
|
||||
public void fetchFromDatanodeException() {}
|
||||
|
||||
public void readFromDatanodeDelay() {}
|
||||
}
|
||||
|
|
|
@ -1044,6 +1044,7 @@ implements ByteBufferReadable, CanSetDropBehind, CanSetReadahead,
|
|||
throw new IOException("truncated return from reader.read(): " +
|
||||
"excpected " + len + ", got " + nread);
|
||||
}
|
||||
DFSClientFaultInjector.get().readFromDatanodeDelay();
|
||||
return;
|
||||
} catch (ChecksumException e) {
|
||||
String msg = "fetchBlockByteRange(). Got a checksum exception for "
|
||||
|
|
|
@ -289,11 +289,12 @@ public class TestPread {
|
|||
DFSClientFaultInjector.instance = Mockito
|
||||
.mock(DFSClientFaultInjector.class);
|
||||
DFSClientFaultInjector injector = DFSClientFaultInjector.instance;
|
||||
final int sleepMs = 100;
|
||||
Mockito.doAnswer(new Answer<Void>() {
|
||||
@Override
|
||||
public Void answer(InvocationOnMock invocation) throws Throwable {
|
||||
if (true) {
|
||||
Thread.sleep(hedgedReadTimeoutMillis + 1);
|
||||
Thread.sleep(hedgedReadTimeoutMillis + sleepMs);
|
||||
if (DFSClientFaultInjector.exceptionNum.compareAndSet(0, 1)) {
|
||||
System.out.println("-------------- throw Checksum Exception");
|
||||
throw new ChecksumException("ChecksumException test", 100);
|
||||
|
@ -302,6 +303,15 @@ public class TestPread {
|
|||
return null;
|
||||
}
|
||||
}).when(injector).fetchFromDatanodeException();
|
||||
Mockito.doAnswer(new Answer<Void>() {
|
||||
@Override
|
||||
public Void answer(InvocationOnMock invocation) throws Throwable {
|
||||
if (true) {
|
||||
Thread.sleep(sleepMs * 2);
|
||||
}
|
||||
return null;
|
||||
}
|
||||
}).when(injector).readFromDatanodeDelay();
|
||||
|
||||
MiniDFSCluster cluster = new MiniDFSCluster.Builder(conf).numDataNodes(2)
|
||||
.format(true).build();
|
||||
|
|
Loading…
Reference in New Issue