You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Bolke de Bruin <bd...@gmail.com> on 2020/03/26 13:42:38 UTC
Review Request 72272: Upgrade and improve Presto plugin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72272/
-----------------------------------------------------------
Review request for ranger.
Bugs: https://issues.apache.org/jira/browse/RANGER-2754
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/RANGER-2754
Repository: ranger
Description
-------
Upgrade and improve Presto plugin
- Presto SQL 331 has changed its security API and has Row level / column masking functionality
- Upgraded Hadoop dependency to 3.1.3 (from 3.1.1) due to improved security handling
- New features like session properties and system properties
Diffs
-----
agents-common/src/main/resources/service-defs/ranger-servicedef-presto.json 56a8f5ac0
distro/src/main/assembly/plugin-presto.xml d2075bfe7
plugin-presto/conf/ranger-presto-security.xml 9feae81a6
plugin-presto/pom.xml b63f7dede
plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java 3ab63f590
plugin-presto/src/test/java/org/apache/ranger/authorization/presto/authorizer/RangerAdminClientImpl.java PRE-CREATION
plugin-presto/src/test/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControlTest.java PRE-CREATION
plugin-presto/src/test/resources/log4j.properties PRE-CREATION
plugin-presto/src/test/resources/presto-policies.json PRE-CREATION
plugin-presto/src/test/resources/ranger-presto-security.xml PRE-CREATION
pom.xml 22926fd7d
ranger-presto-plugin-shim/pom.xml d8ff88d0f
ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerConfig.java 67b0d2434
ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java e89f646e1
Diff: https://reviews.apache.org/r/72272/diff/1/
Testing
-------
- New Unit tests added
- Tested locally in production
Thanks,
Bolke de Bruin
Re: Review Request 72272: Upgrade and improve Presto plugin
Posted by Bolke de Bruin <bd...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72272/
-----------------------------------------------------------
(Updated March 26, 2020, 2:07 p.m.)
Review request for ranger.
Bugs: https://issues.apache.org/jira/browse/RANGER-2754
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/RANGER-2754
Repository: ranger
Description
-------
Upgrade and improve Presto plugin
- Presto SQL 331 has changed its security API and has Row level / column masking functionality
- Upgraded Hadoop dependency to 3.1.3 (from 3.1.1) due to improved security handling
- New features like session properties and system properties
Diffs (updated)
-----
agents-common/src/main/resources/service-defs/ranger-servicedef-presto.json 56a8f5ac0
distro/src/main/assembly/plugin-presto.xml d2075bfe7
plugin-presto/pom.xml b63f7dede
plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java 3ab63f590
plugin-presto/src/test/java/org/apache/ranger/authorization/presto/authorizer/RangerAdminClientImpl.java PRE-CREATION
plugin-presto/src/test/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControlTest.java PRE-CREATION
plugin-presto/src/test/resources/log4j.properties PRE-CREATION
plugin-presto/src/test/resources/presto-policies.json PRE-CREATION
plugin-presto/src/test/resources/ranger-presto-security.xml PRE-CREATION
pom.xml 22926fd7d
ranger-presto-plugin-shim/pom.xml d8ff88d0f
ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerConfig.java 67b0d2434
ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java e89f646e1
Diff: https://reviews.apache.org/r/72272/diff/2/
Changes: https://reviews.apache.org/r/72272/diff/1-2/
Testing
-------
- New Unit tests added
- Tested locally in production
Thanks,
Bolke de Bruin