You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Qiang Zhang <zh...@zte.com.cn> on 2017/08/08 02:08:06 UTC
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61021/
-----------------------------------------------------------
(Updated 八月 8, 2017, 2:08 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.Add "Project" as a another resource in the resource definition
2.Remove search user by calling kylin rest api
3.Optimized authentication(checkPermission)
4.Ohter small modifications
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/service/RangerBaseService.java f1c6b9f
agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java 0bc09f6
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 01005b7
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
security-admin/src/main/webapp/scripts/utils/XAEnums.js 164e939
security-admin/src/main/webapp/scripts/utils/XAUtils.js 1979847
security-admin/src/main/webapp/scripts/views/policies/PermissionList.js 067bf3b
src/main/assembly/admin-web.xml cb1aad2
src/main/assembly/plugin-kylin.xml PRE-CREATION
Diff: https://reviews.apache.org/r/61021/diff/2/
Changes: https://reviews.apache.org/r/61021/diff/1-2/
Testing
-------
Tested
Thanks,
Qiang Zhang
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
Posted by Zsombor Gegesy <gz...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61021/#review182699
-----------------------------------------------------------
security-admin/src/main/webapp/scripts/views/policies/PermissionList.js
Lines 195 (patched)
<https://reviews.apache.org/r/61021/#comment258680>
Probably a much bigger change, but ideally, in the web module, there shouldn't be much any service or kylin specific thing.
It would be better to put these kylin roles in the service defs, under for example 'extra options' or some other structure. So next time, when a new service would need a similar functionality, it could reuse the code - and not in a 'copy-paste' sense :)
- Zsombor Gegesy
On Aug. 8, 2017, 2:08 a.m., Qiang Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61021/
> -----------------------------------------------------------
>
> (Updated Aug. 8, 2017, 2:08 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.
>
>
> 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
> -----
>
> agents-common/scripts/enable-agent.sh d31a264
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java f1c6b9f
> agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java 0bc09f6
> 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 01005b7
> 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
> security-admin/src/main/webapp/scripts/utils/XAEnums.js 164e939
> security-admin/src/main/webapp/scripts/utils/XAUtils.js 1979847
> security-admin/src/main/webapp/scripts/views/policies/PermissionList.js 067bf3b
> src/main/assembly/admin-web.xml cb1aad2
> src/main/assembly/plugin-kylin.xml PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/61021/diff/2/
>
>
> Testing
> -------
>
> Tested
>
>
> Thanks,
>
> Qiang Zhang
>
>
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
Posted by Nitin Galave <ni...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61021/#review182681
-----------------------------------------------------------
security-admin/src/main/webapp/scripts/views/policies/PermissionList.js
Lines 196 (patched)
<https://reviews.apache.org/r/61021/#comment258670>
Suggestion: Can you provide options to select2 based on service definition is kylin or not instead of defining select2 two times?
like select2(options);
security-admin/src/main/webapp/scripts/views/policies/PermissionList.js
Lines 201 (patched)
<https://reviews.apache.org/r/61021/#comment258669>
As per changes look like for `Role` drop-down in the policy item:
`Select Role` drop-down allows the user to add a new role.(i.e user defined role)
Is this requirement?
- Nitin Galave
On Aug. 8, 2017, 2:08 a.m., Qiang Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61021/
> -----------------------------------------------------------
>
> (Updated Aug. 8, 2017, 2:08 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.
>
>
> 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
> -----
>
> agents-common/scripts/enable-agent.sh d31a264
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java f1c6b9f
> agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java 0bc09f6
> 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 01005b7
> 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
> security-admin/src/main/webapp/scripts/utils/XAEnums.js 164e939
> security-admin/src/main/webapp/scripts/utils/XAUtils.js 1979847
> security-admin/src/main/webapp/scripts/views/policies/PermissionList.js 067bf3b
> src/main/assembly/admin-web.xml cb1aad2
> src/main/assembly/plugin-kylin.xml PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/61021/diff/2/
>
>
> Testing
> -------
>
> Tested
>
>
> Thanks,
>
> Qiang Zhang
>
>
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
Posted by Qiang Zhang <zh...@zte.com.cn>.
> On 八月 25, 2017, 7:24 a.m., Madhan Neethiraj wrote:
> > Qiang Zhang - it is good to see addition of new Ranger plugin. Good work! Please go through review comments. Overall, the plugin implementation looks good.
Ok. Thanks.
- Qiang
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61021/#review183830
-----------------------------------------------------------
On 八月 8, 2017, 2:08 a.m., Qiang Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61021/
> -----------------------------------------------------------
>
> (Updated 八月 8, 2017, 2:08 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.
>
>
> 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
> -----
>
> agents-common/scripts/enable-agent.sh d31a264
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java f1c6b9f
> agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java 0bc09f6
> 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 01005b7
> 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
> security-admin/src/main/webapp/scripts/utils/XAEnums.js 164e939
> security-admin/src/main/webapp/scripts/utils/XAUtils.js 1979847
> security-admin/src/main/webapp/scripts/views/policies/PermissionList.js 067bf3b
> src/main/assembly/admin-web.xml cb1aad2
> src/main/assembly/plugin-kylin.xml PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/61021/diff/2/
>
>
> Testing
> -------
>
> Tested
>
>
> Thanks,
>
> Qiang Zhang
>
>
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61021/#review183830
-----------------------------------------------------------
Qiang Zhang - it is good to see addition of new Ranger plugin. Good work! Please go through review comments. Overall, the plugin implementation looks good.
plugin-kylin/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java
Lines 95 (patched)
<https://reviews.apache.org/r/61021/#comment259902>
Looks like auditHandler will never be null here, given its initialization above in line #81. Consider moving line #96 inside the 'if' block that ends at #93.
plugin-kylin/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java
Lines 109 (patched)
<https://reviews.apache.org/r/61021/#comment259903>
This will get the IP address of the host where the plugin runs. Is this intentional? If yes, perhaps you can get this value during plugin initalization and use it for all request. This is unlikley to change across authorization requests.
plugin-kylin/src/main/java/org/apache/ranger/authorization/kylin/authorizer/RangerKylinAuthorizer.java
Lines 167 (patched)
<https://reviews.apache.org/r/61021/#comment259904>
RangerKylinAuditHandler doesn't seem to handle processResult() differently from RangerDefaultAuditHandler. You can remove this class and instead use RangerDefaultAuditHandler set in line #135. With this, you don't need to pass auditHandler argument in isAccessAllowed() call - line #89.
- Madhan Neethiraj
On Aug. 8, 2017, 2:08 a.m., Qiang Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61021/
> -----------------------------------------------------------
>
> (Updated Aug. 8, 2017, 2:08 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.
>
>
> 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
> -----
>
> agents-common/scripts/enable-agent.sh d31a264
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java f1c6b9f
> agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java 0bc09f6
> 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 01005b7
> 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
> security-admin/src/main/webapp/scripts/utils/XAEnums.js 164e939
> security-admin/src/main/webapp/scripts/utils/XAUtils.js 1979847
> security-admin/src/main/webapp/scripts/views/policies/PermissionList.js 067bf3b
> src/main/assembly/admin-web.xml cb1aad2
> src/main/assembly/plugin-kylin.xml PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/61021/diff/2/
>
>
> Testing
> -------
>
> Tested
>
>
> Thanks,
>
> Qiang Zhang
>
>
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
Posted by Qiang Zhang <zh...@zte.com.cn>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61021/#review186196
-----------------------------------------------------------
@Colm, Zsombor Gegesy, Ramesh Mani,Nitin Galave, Pradeep Agrawal, Zsombor Gegesy and Madhan Neethiraj, the https://issues.apache.org/jira/browse/KYLIN-2703 had been resolved. So our RANGER-1672 can be resolved now. We have rigorously tested RANGER-1672. Thanks.
- Qiang Zhang
On 九月 22, 2017, 7:48 a.m., Qiang Zhang wrote:
>
> -----------------------------------------------------------
> 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.
>
>
> 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
> -----
>
> 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/
>
>
> Testing
> -------
>
> Tested
>
>
> Thanks,
>
> Qiang Zhang
>
>
Re: Review Request 61021: RANGER-1672:Ranger supports plugin to
enable, monitor and manage apache kylin
Posted by Qiang Zhang <zh...@zte.com.cn>.
-----------------------------------------------------------
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