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

Ayush Saxena commented on HDFS-14848:
-------------------------------------

Thanx [~akki261001] for putting this up. 
Actually the reason for ACE is :

{code:java}
    try {
      checkTraverse(pc, iip, dirOp);
    } catch (ParentNotDirectoryException pnde) {
      if (!isCreate) {
        throw new AccessControlException(pnde.getMessage());
      }
      throw pnde;
    }
{code}

Ideally it was to prevent {{ParentNotDirectoryException}}, but ACE doesn't 
seems to be an ideal substitute to it. Changing to FileNotFoundException would 
be a better choice, it could atleast be near the context. Anyway ACE has 
different handling too, as in {{FsNamesystem}} for audit logging only ACE is 
caught, so here unnecessarily this will also get caught.

will upload patch changinf to FNF.


> ACE for Non Existing Paths
> --------------------------
>
>                 Key: HDFS-14848
>                 URL: https://issues.apache.org/jira/browse/HDFS-14848
>             Project: Hadoop HDFS
>          Issue Type: Bug
>    Affects Versions: 3.3.0
>            Reporter: Akshay Agarwal
>            Assignee: Ayush Saxena
>            Priority: Major
>
> Access control exception for several operations in some cases when the path 
> doesn't exist.
> For eg: SetStoragePolicy, getStoragePolicy, getBlockLocations and many other.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to