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

Eli Collins commented on HADOOP-9178:
-------------------------------------

bq. I removed the deprecated ones - should I leave them in?

Removing them is correct. Two of them already have deprecations added to 
ConfigUtil.java, there's no equivalent deprecation for 
security.inter.tracker.protocol.acl (security.containermanager.protocol.acl is 
closest but since it's yarn only probably doesn't need to be added as a 
deprecation here).

Patch looks good, thanks for all the cleanup!  One nit: three of the configs 
(security.ha.service.protocol.acl, security.zkfc.protocol.acl, are 
security.qjournal.service.protocol.acl) in the yarn section of 
hadoop-policy.xml, please move them up next to the hdfs configs and also fix 
the formatting of security.mrhs.client.protocol.acl.
                
> src/main/conf is missing hadoop-policy.xml
> ------------------------------------------
>
>                 Key: HADOOP-9178
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9178
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 2.0.2-alpha
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>            Priority: Minor
>             Fix For: 2.0.3-alpha
>
>         Attachments: HADOOP-9178-1.patch, HADOOP-9178-1.patch, 
> HADOOP-9178.patch
>
>
> src/main/conf contains hadoop-env.sh and core-site.xml, but is missing 
> hadoop-policy.xml

--
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