svn merge -c 1402270 from trunk for HDFS-3948. Do not use hflush in TestWebHDFS.testNamenodeRestart() since the out stream returned by WebHdfsFileSystem does not support it.
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1402273 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
1711ef5bfb
commit
6f91542707
|
@ -150,6 +150,10 @@ Release 2.0.3-alpha - Unreleased
|
|||
|
||||
HDFS-4022. Replication not happening for appended block. (Vinay via umamahesh)
|
||||
|
||||
HDFS-3948. Do not use hflush in TestWebHDFS.testNamenodeRestart() since the
|
||||
out stream returned by WebHdfsFileSystem does not support it. (Jing Zhao
|
||||
via szetszwo)
|
||||
|
||||
Release 2.0.2-alpha - 2012-09-07
|
||||
|
||||
INCOMPATIBLE CHANGES
|
||||
|
|
|
@ -824,7 +824,14 @@ public class TestDFSClientRetries {
|
|||
new Random().nextBytes(bytes);
|
||||
out4.write(bytes);
|
||||
out4.write(bytes);
|
||||
out4.hflush();
|
||||
if (isWebHDFS) {
|
||||
// WebHDFS does not support hflush. To avoid DataNode communicating with
|
||||
// NN while we're shutting down NN, we call out4.close() to finish
|
||||
// writing the data
|
||||
out4.close();
|
||||
} else {
|
||||
out4.hflush();
|
||||
}
|
||||
|
||||
//shutdown namenode
|
||||
assertTrue(HdfsUtils.isHealthy(uri));
|
||||
|
@ -837,10 +844,12 @@ public class TestDFSClientRetries {
|
|||
public void run() {
|
||||
try {
|
||||
//write some more data and then close the file
|
||||
out4.write(bytes);
|
||||
out4.write(bytes);
|
||||
out4.write(bytes);
|
||||
out4.close();
|
||||
if (!isWebHDFS) {
|
||||
out4.write(bytes);
|
||||
out4.write(bytes);
|
||||
out4.write(bytes);
|
||||
out4.close();
|
||||
}
|
||||
} catch (Exception e) {
|
||||
exceptions.add(e);
|
||||
}
|
||||
|
@ -923,7 +932,11 @@ public class TestDFSClientRetries {
|
|||
Assert.assertEquals(String.format("count=%d", count),
|
||||
bytes[count % bytes.length], (byte)r);
|
||||
}
|
||||
Assert.assertEquals(5 * bytes.length, count);
|
||||
if (!isWebHDFS) {
|
||||
Assert.assertEquals(5 * bytes.length, count);
|
||||
} else {
|
||||
Assert.assertEquals(2 * bytes.length, count);
|
||||
}
|
||||
in.close();
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue