You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2021/05/25 08:51:03 UTC
Review Request 73374: RANGER-3296: updated getResourceACLs() to
include data-masks and row-filters as well
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73374/
-----------------------------------------------------------
Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
Bugs: RANGER-3296
https://issues.apache.org/jira/browse/RANGER-3296
Repository: ranger
Description
-------
- added fields rowFilters and dataMasks to RangerResourceACLs
- updated RangerBasePlugin.getResourceACLs() to populate above new fields
Diffs
-----
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicy.java 3a6f416b3
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java 75811b8e2
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceACLs.java a0def5491
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java 2edd992cf
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyEvaluator.java c611b5e08
agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyACLs.java e1709ad94
agents-common/src/test/resources/policyengine/ACLResourceTags.json 711190c9e
agents-common/src/test/resources/policyengine/test_aclprovider_data_mask.json PRE-CREATION
Diff: https://reviews.apache.org/r/73374/diff/1/
Testing
-------
- added unit tests to cover data-mask and row-filter use cases
Thanks,
Madhan Neethiraj
Re: Review Request 73374: RANGER-3296: updated getResourceACLs() to
include data-masks and row-filters as well
Posted by Abhay Kulkarni <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73374/#review223045
-----------------------------------------------------------
Ship it!
Ship It!
- Abhay Kulkarni
On May 25, 2021, 8:51 a.m., Madhan Neethiraj wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73374/
> -----------------------------------------------------------
>
> (Updated May 25, 2021, 8:51 a.m.)
>
>
> Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
>
>
> Bugs: RANGER-3296
> https://issues.apache.org/jira/browse/RANGER-3296
>
>
> Repository: ranger
>
>
> Description
> -------
>
> - added fields rowFilters and dataMasks to RangerResourceACLs
> - updated RangerBasePlugin.getResourceACLs() to populate above new fields
>
>
> Diffs
> -----
>
> agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicy.java 3a6f416b3
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java 75811b8e2
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceACLs.java a0def5491
> agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java 2edd992cf
> agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyEvaluator.java c611b5e08
> agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyACLs.java e1709ad94
> agents-common/src/test/resources/policyengine/ACLResourceTags.json 711190c9e
> agents-common/src/test/resources/policyengine/test_aclprovider_mask_filter.json PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/73374/diff/3/
>
>
> Testing
> -------
>
> - added unit tests to cover data-mask and row-filter use cases
>
>
> Thanks,
>
> Madhan Neethiraj
>
>