You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Abhay Kulkarni <ak...@hortonworks.com> on 2017/04/05 20:49:39 UTC
Review Request 58219: Support administration of Tag-based masking
policies
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58219/
-----------------------------------------------------------
Review request for ranger, Madhan Neethiraj and Nitin Galave.
Bugs: RANGER-1494
https://issues.apache.org/jira/browse/RANGER-1494
Repository: ranger
Description
-------
Supported modification of tag servicedef to reflect changes to non-tag based service definition.
Diffs
-----
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java 0793a6a
agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractServiceStore.java 1702dec
Diff: https://reviews.apache.org/r/58219/diff/1/
Testing
-------
Created/Deleted/Updated non-tag service-def with dataMaskDef modifications. Ensured they are correctly reflected in the tag service-def.
Thanks,
Abhay Kulkarni
Re: Review Request 58219: Support administration of Tag-based masking
policies
Posted by Colm O hEigeartaigh <co...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58219/#review173179
-----------------------------------------------------------
There are several instances where you are creating an Iterator which is just debug logging, meaning that if debug logging is turned off we are doing a load of empty iterations, e.g.:
+ for (RangerServiceDef.RangerDataMaskTypeDef maskTypeDef : maskTypesToAdd) {
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("maskTypeDef-to-add:[" + maskTypeDef + "]");
+ }
+ }
+
Much better to instead put the if statement outside the for loop.
- Colm O hEigeartaigh
On April 21, 2017, 9:23 p.m., Abhay Kulkarni wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58219/
> -----------------------------------------------------------
>
> (Updated April 21, 2017, 9:23 p.m.)
>
>
> Review request for ranger, Madhan Neethiraj and Nitin Galave.
>
>
> Bugs: RANGER-1494
> https://issues.apache.org/jira/browse/RANGER-1494
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Supported modification of tag servicedef to reflect changes to non-tag based service definition.
>
>
> Diffs
> -----
>
> agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceDef.java 7b3bd95
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java 0793a6a
> agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractServiceStore.java 1702dec
>
>
> Diff: https://reviews.apache.org/r/58219/diff/2/
>
>
> Testing
> -------
>
> Created/Deleted/Updated non-tag service-def with dataMaskDef modifications. Ensured they are correctly reflected in the tag service-def.
>
>
> Thanks,
>
> Abhay Kulkarni
>
>
Re: Review Request 58219: Support administration of Tag-based masking
policies
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58219/#review176563
-----------------------------------------------------------
Ship it!
Ship It!
- Madhan Neethiraj
On April 21, 2017, 9:23 p.m., Abhay Kulkarni wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58219/
> -----------------------------------------------------------
>
> (Updated April 21, 2017, 9:23 p.m.)
>
>
> Review request for ranger, Madhan Neethiraj and Nitin Galave.
>
>
> Bugs: RANGER-1494
> https://issues.apache.org/jira/browse/RANGER-1494
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Supported modification of tag servicedef to reflect changes to non-tag based service definition.
>
>
> Diffs
> -----
>
> agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceDef.java 7b3bd95
> agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java 0793a6a
> agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractServiceStore.java 1702dec
>
>
> Diff: https://reviews.apache.org/r/58219/diff/2/
>
>
> Testing
> -------
>
> Created/Deleted/Updated non-tag service-def with dataMaskDef modifications. Ensured they are correctly reflected in the tag service-def.
>
>
> Thanks,
>
> Abhay Kulkarni
>
>
Re: Review Request 58219: Support administration of Tag-based masking
policies
Posted by Abhay Kulkarni <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58219/
-----------------------------------------------------------
(Updated April 21, 2017, 9:23 p.m.)
Review request for ranger, Madhan Neethiraj and Nitin Galave.
Changes
-------
Added row-filter processing for tag service. Refactored to remove duplicate code.
Bugs: RANGER-1494
https://issues.apache.org/jira/browse/RANGER-1494
Repository: ranger
Description
-------
Supported modification of tag servicedef to reflect changes to non-tag based service definition.
Diffs (updated)
-----
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceDef.java 7b3bd95
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java 0793a6a
agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractServiceStore.java 1702dec
Diff: https://reviews.apache.org/r/58219/diff/2/
Changes: https://reviews.apache.org/r/58219/diff/1-2/
Testing
-------
Created/Deleted/Updated non-tag service-def with dataMaskDef modifications. Ensured they are correctly reflected in the tag service-def.
Thanks,
Abhay Kulkarni