-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68006/#review206322
-----------------------------------------------------------


Ship it!




Ship It!

- Madhan Neethiraj


On July 21, 2018, 1:56 a.m., Abhay Kulkarni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68006/
> -----------------------------------------------------------
> 
> (Updated July 21, 2018, 1:56 a.m.)
> 
> 
> Review request for ranger, Madhan Neethiraj and Ramesh Mani.
> 
> 
> Bugs: RANGER-2158
>     https://issues.apache.org/jira/browse/RANGER-2158
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Performance improvement to REST API call to update policies. Currently,
> 
> 1) the update is a bit slower because of the repeated calls to DB for policy 
> objects.
> 
> 2) policy cache need not be built while updating the policy via REST API as 
> this call if it happened to be in the timeframe of the policy download,  it 
> is slowing down the update process.
> 
> 
> Diffs
> -----
> 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerPolicyValidator.java
>  4d2fa23ad 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidator.java
>  f2f5977fa 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/store/ServiceStore.java 
> 2c57a6fd8 
>   
> agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerPolicyValidator.java
>  d8a135430 
>   
> agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerValidator.java
>  a749b27d6 
>   security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 
> f00d31185 
>   
> security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java
>  519d8e94d 
> 
> 
> Diff: https://reviews.apache.org/r/68006/diff/1/
> 
> 
> Testing
> -------
> 
> Ran all unit tests successfully.
> 
> 
> Thanks,
> 
> Abhay Kulkarni
> 
>

Reply via email to