You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Pradeep Agrawal <pr...@gmail.com> on 2020/02/24 09:16:35 UTC
Re: Review Request 72030: RANGER-2706 usermapsyncenabled can be
removed from code
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72030/#review219634
-----------------------------------------------------------
Ship it!
Ship It!
- Pradeep Agrawal
On Jan. 21, 2020, 10:09 a.m., Lars Francke wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72030/
> -----------------------------------------------------------
>
> (Updated Jan. 21, 2020, 10:09 a.m.)
>
>
> Review request for ranger.
>
>
> Bugs: RANGER-2706
> https://issues.apache.org/jira/browse/RANGER-2706
>
>
> Repository: ranger
>
>
> Description
> -------
>
> RANGER-742 removed the need for the usermapsyncenabled property but it's still being read, printed in the logs and I believe it can even be configured in Ambari but all it does is confuse so this should be removed.
>
>
> Diffs
> -----
>
> migration-util/ambari2.1-hdp2.3-ranger0.50/bin/import_ranger_to_ambari.py e25b8ae89
> ugsync/ldapconfigchecktool/ldapconfigcheck/conf/input.properties 449ee6f44
> ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/LdapConfig.java 920ebf0b3
> ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapDeltaUserGroupBuilder.java ca50f098a
> ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java 8efa1613a
> ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java 1d4e37fcf
> ugsync/src/test/resources/ranger-ugsync-site.xml 0a1a86da0
> unixauthservice/scripts/templates/installprop2xml.properties fa342fbff
> unixauthservice/scripts/templates/ranger-ugsync-template.xml 0c2d1fcac
>
>
> Diff: https://reviews.apache.org/r/72030/diff/1/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Lars Francke
>
>