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

Arpit Agarwal commented on HDFS-9415:
-------------------------------------

Thanks for the updated patch [~xiaobingo].

# I think you can remove this sentence _A comma separated list of groups is not 
allowed. e.g., "supergroup" is good, but "supergroup1,supergroup2" is 
incorrect._ Just stating it should be a single group name is sufficient.
# _The format are_ should be the _The format is_.
# You can just convert the numbered list in the {{dfs.cluster.administrators}} 
description to paragraphs. I am not sure the numbered lists will render well 
when the XML is convered to HTML.
# I did not mention the wildcard option for {{dfs.cluster.administrators}} as 
IMO it is best left undocumented. If you choose to include it as Daniel 
suggests, see {{AccessControlList#buildACL}} for the precise behavior of the 
wildcard.

> Document dfs.cluster.administrators and dfs.permissions.superusergroup
> ----------------------------------------------------------------------
>
>                 Key: HDFS-9415
>                 URL: https://issues.apache.org/jira/browse/HDFS-9415
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: documentation
>            Reporter: Arpit Agarwal
>            Assignee: Xiaobing Zhou
>         Attachments: HDFS-9415.001.patch, HDFS-9415.002.patch
>
>
> dfs.cluster.administrators and dfs.permissions.superusergroup documentation 
> is not clear enough.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to