You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by in...@apache.org on 2018/05/18 02:29:27 UTC
[4/5] hadoop git commit: HDFS-13586. Fsync fails on directories on
Windows. Contributed by Lukas Majercack.
HDFS-13586. Fsync fails on directories on Windows. Contributed by Lukas Majercack.
(cherry picked from commit 8783613696674aba4ae1739c6e8f48cda0d1c386)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/c4d4ae62
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/c4d4ae62
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/c4d4ae62
Branch: refs/heads/branch-2
Commit: c4d4ae62cb94189ad2a37a6c3c47d01abf592cbb
Parents: a5ec44c
Author: Inigo Goiri <in...@apache.org>
Authored: Thu May 17 19:26:44 2018 -0700
Committer: Inigo Goiri <in...@apache.org>
Committed: Thu May 17 19:28:28 2018 -0700
----------------------------------------------------------------------
.../src/main/java/org/apache/hadoop/io/IOUtils.java | 7 +++++++
1 file changed, 7 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/c4d4ae62/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/IOUtils.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/IOUtils.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/IOUtils.java
index 03c488c..547a5b4 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/IOUtils.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/IOUtils.java
@@ -398,6 +398,13 @@ public class IOUtils {
"File/Directory " + fileToSync.getAbsolutePath() + " does not exist");
}
boolean isDir = fileToSync.isDirectory();
+
+ // HDFS-13586, FileChannel.open fails with AccessDeniedException
+ // for any directory, ignore.
+ if (isDir && Shell.WINDOWS) {
+ return;
+ }
+
// If the file is a directory we have to open read-only, for regular files
// we must open r/w for the fsync to have an effect. See
// http://blog.httrack.com/blog/2013/11/15/
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org