[ 
https://issues.apache.org/jira/browse/HDFS-5899?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris Nauroth updated HDFS-5899:
--------------------------------

    Attachment: HDFS-5899.2.patch

Arpit and Nicholas, thank you for the reviews.  I'm uploading patch version 2, 
which incorporates Arpit's suggestion to put more information into 
hdfs-default.xml.

bq. Some thoughts: It would be good if the conf checks are moved to AclStorage 
so that the checking becomes centralized.

I didn't put this in {{AclStorage}}, because the {{AclStorage}} methods only 
get called while holding the namesystem write lock.  If ACLs are disabled, then 
I'd prefer to reject API calls earlier than that so that there is no impact on 
the system, no matter how small.

Nicholas, are you still OK if we go with this version of the patch?  If so, 
then I'll plan to commit the v2 patch over the weekend.

> Add configuration flag to disable/enable support for ACLs.
> ----------------------------------------------------------
>
>                 Key: HDFS-5899
>                 URL: https://issues.apache.org/jira/browse/HDFS-5899
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: namenode
>    Affects Versions: HDFS ACLs (HDFS-4685)
>            Reporter: Chris Nauroth
>            Assignee: Chris Nauroth
>         Attachments: HDFS-5899.1.patch, HDFS-5899.2.patch
>
>
> Add a new configuration property that allows administrators to toggle support 
> for HDFS ACLs on/off.  By default, the flag will be off.  This is a 
> conservative choice, and administrators interested in using ACLs can enable 
> it explicitly.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to