[ 
https://issues.apache.org/jira/browse/HADOOP-8973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13592918#comment-13592918
 ] 

Ivan Mitic commented on HADOOP-8973:
------------------------------------

Thanks Chris for the patch. I am fine with the current patch, but would still 
like to reiterate on Bikas' question on why we cannot use 
impliesRead/impliesWrite etc. At this point, we already assume the POSIX like 
semantic in Hadoop across the board (this is why we have winutils). Leveriging 
winutils in CheckDisk would provide a nice symmetry in the test.

You bring up a good point about not completely covering the NTFS ACLs space. I 
think this is fine for now. Eventually, as we improve, we can think of good 
ways to abstract this out, but this goes way beyond today.

Let me know what you think.


                
> DiskChecker cannot reliably detect an inaccessible disk on Windows with NTFS 
> ACLs
> ---------------------------------------------------------------------------------
>
>                 Key: HADOOP-8973
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8973
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: util
>    Affects Versions: trunk-win
>            Reporter: Chris Nauroth
>            Assignee: Chris Nauroth
>         Attachments: HADOOP-8973-branch-trunk-win.patch
>
>
> DiskChecker.checkDir uses File.canRead, File.canWrite, and File.canExecute to 
> check if a directory is inaccessible.  These APIs are not reliable on Windows 
> with NTFS ACLs due to a known JVM bug.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to