----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61021/ -----------------------------------------------------------
(Updated 九月 22, 2017, 7:48 a.m.) Review request for ranger, Ankita Sinha, Don Bosco Durai, Colm O hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, sam rome, and Velmurugan Periasamy. Changes ------- Fixed patch according to the reviews, major modifications: 1.Remove the special processing of Group(Role) and Permissions for kylin in the create/edit policy web page. Kylin use LDAP/AD authentication integration and support the Hadoop user-group mapping facility. 2.Remove the special processing of Group(Role) for kylin in the Java background. 3.Based on the refactored authorization interfaces of Kylin, modify Ranger Kylin Plugin implements to fit changes. 4.Kylin authorization interfaces is available in Kylin master with version 2.2.0-SNAPSHOT, you can build kylin to test this patch manually, or wait kylin next release. 5.Ohter small modifications for optimization. Bugs: RANGER-1672 https://issues.apache.org/jira/browse/RANGER-1672 Repository: ranger Description ------- Ranger supports plugin to enable, monitor and manage apache kylin Diffs (updated) ----- agents-common/scripts/enable-agent.sh d31a264 agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java 9463ab8 agents-common/src/main/java/org/apache/ranger/plugin/util/PasswordUtils.java 58cdd35 agents-common/src/main/resources/service-defs/ranger-servicedef-kylin.json PRE-CREATION plugin-kylin/.gitignore PRE-CREATION plugin-kylin/conf/ranger-kylin-audit-changes.cfg PRE-CREATION plugin-kylin/conf/ranger-kylin-audit.xml PRE-CREATION plugin-kylin/conf/ranger-kylin-security-changes.cfg PRE-CREATION plugin-kylin/conf/ranger-kylin-security.xml PRE-CREATION plugin-kylin/conf/ranger-policymgr-ssl-changes.cfg PRE-CREATION plugin-kylin/conf/ranger-policymgr-ssl.xml PRE-CREATION plugin-kylin/pom.xml PRE-CREATION plugin-kylin/scripts/install.properties PRE-CREATION plugin-kylin/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java PRE-CREATION plugin-kylin/src/main/java/org/apache/ranger/services/kylin/RangerServiceKylin.java PRE-CREATION plugin-kylin/src/main/java/org/apache/ranger/services/kylin/client/KylinClient.java PRE-CREATION plugin-kylin/src/main/java/org/apache/ranger/services/kylin/client/KylinResourceMgr.java PRE-CREATION plugin-kylin/src/main/java/org/apache/ranger/services/kylin/client/json/model/KylinCubeResponse.java PRE-CREATION plugin-kylin/src/main/java/org/apache/ranger/services/kylin/client/json/model/KylinProjectResponse.java PRE-CREATION pom.xml 3958014 ranger-kylin-plugin-shim/.gitignore PRE-CREATION ranger-kylin-plugin-shim/pom.xml PRE-CREATION ranger-kylin-plugin-shim/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java PRE-CREATION src/main/assembly/admin-web.xml 0e97818 src/main/assembly/plugin-kylin.xml PRE-CREATION Diff: https://reviews.apache.org/r/61021/diff/3/ Changes: https://reviews.apache.org/r/61021/diff/2-3/ Testing ------- Tested Thanks, Qiang Zhang