Repository: hadoop
Updated Branches:
  refs/heads/branch-2 a5ec44c65 -> c4d4ae62c
  refs/heads/branch-2.9 905c37734 -> 8e5b5a108
  refs/heads/branch-3.0 cdc16b3a2 -> 7336d0123
  refs/heads/branch-3.1 079178557 -> 472495ed1
  refs/heads/trunk 2f2dd22aa -> 878361369


HDFS-13586. Fsync fails on directories on Windows. Contributed by Lukas 
Majercack.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/87836136
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/87836136
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/87836136

Branch: refs/heads/trunk
Commit: 8783613696674aba4ae1739c6e8f48cda0d1c386
Parents: 2f2dd22
Author: Inigo Goiri <inigo...@apache.org>
Authored: Thu May 17 19:26:44 2018 -0700
Committer: Inigo Goiri <inigo...@apache.org>
Committed: Thu May 17 19:26:44 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/87836136/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 f451ff3..7288812 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
@@ -414,6 +414,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-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-commits-h...@hadoop.apache.org

Reply via email to