----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66589/ -----------------------------------------------------------
(Updated April 13, 2018, 1:31 p.m.) Review request for atlas, Ashutosh Mestry and Madhan Neethiraj. Changes ------- This patch handles the review comments from Madhan N. Bugs: ATLAS-2557 https://issues.apache.org/jira/browse/ATLAS-2557 Repository: atlas Description ------- This patch includes fix to allow to lookup hadoop ldap groups when group from are wrongly set. In this case reading groups from hadoop ldap group mapping can help to get all groups. Diffs (updated) ----- intg/src/main/java/org/apache/atlas/utils/AuthenticationUtil.java 09d8085c1 webapp/src/main/java/org/apache/atlas/web/security/AtlasAbstractAuthenticationProvider.java f77cb01d1 Diff: https://reviews.apache.org/r/66589/diff/2/ Changes: https://reviews.apache.org/r/66589/diff/1-2/ Testing ------- Tested with property atlas.authentication.ugi-groups.include-hadoop-groups set true and false. to verify the groups return are correct. Thanks, Nixon Rodrigues