HDFS-5268. NFS write commit verifier is not set in a few places. Contributed by Brandon Li
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1527087 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
58b08e11b9
commit
027419832c
|
@ -422,7 +422,7 @@ class OpenFileCtx {
|
|||
LOG.warn("Haven't noticed any partial overwrite for a sequential file"
|
||||
+ " write requests. Treat it as a real random write, no support.");
|
||||
response = new WRITE3Response(Nfs3Status.NFS3ERR_INVAL, wccData, 0,
|
||||
WriteStableHow.UNSTABLE, 0);
|
||||
WriteStableHow.UNSTABLE, Nfs3Constant.WRITE_COMMIT_VERF);
|
||||
} else {
|
||||
if (LOG.isDebugEnabled()) {
|
||||
LOG.debug("Process perfectOverWrite");
|
||||
|
@ -559,7 +559,7 @@ class OpenFileCtx {
|
|||
if (comparator.compare(readbuffer, 0, readCount, data, 0, count) != 0) {
|
||||
LOG.info("Perfect overwrite has different content");
|
||||
response = new WRITE3Response(Nfs3Status.NFS3ERR_INVAL, wccData, 0,
|
||||
stableHow, 0);
|
||||
stableHow, Nfs3Constant.WRITE_COMMIT_VERF);
|
||||
} else {
|
||||
LOG.info("Perfect overwrite has same content,"
|
||||
+ " updating the mtime, then return success");
|
||||
|
@ -571,12 +571,12 @@ class OpenFileCtx {
|
|||
LOG.info("Got error when processing perfect overwrite, path=" + path
|
||||
+ " error:" + e);
|
||||
return new WRITE3Response(Nfs3Status.NFS3ERR_IO, wccData, 0, stableHow,
|
||||
0);
|
||||
Nfs3Constant.WRITE_COMMIT_VERF);
|
||||
}
|
||||
|
||||
wccData.setPostOpAttr(postOpAttr);
|
||||
response = new WRITE3Response(Nfs3Status.NFS3_OK, wccData, count,
|
||||
stableHow, 0);
|
||||
stableHow, Nfs3Constant.WRITE_COMMIT_VERF);
|
||||
}
|
||||
return response;
|
||||
}
|
||||
|
|
|
@ -354,6 +354,8 @@ Release 2.1.2 - UNRELEASED
|
|||
HDFS-5186. TestFileJournalManager fails on Windows due to file handle leaks.
|
||||
(Chuan Liu via cnauroth)
|
||||
|
||||
HDFS-5268. NFS write commit verifier is not set in a few places (brandonli)
|
||||
|
||||
Release 2.1.1-beta - 2013-09-23
|
||||
|
||||
INCOMPATIBLE CHANGES
|
||||
|
|
Loading…
Reference in New Issue