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 2019/11/22 17:43:58 UTC

Review Request 71808: RANGER-2652: refactor policy-engine #2 - renames, whitespace updates

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

Review request for ranger and Abhay Kulkarni.


Repository: ranger


Description
-------

RangerPolicyEngineCache.java                 => RangerPolicyAdminCache.java
RangerPolicyEngineCacheForEngineOptions.java => RangerPolicyAdminCacheForEngineOptions.java


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/PolicyEngine.java b24d37c5a 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessRequestProcessor.java a6836998d 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java afe6683fb 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java 28b441a44 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java 11c1eeb98 
  agents-common/src/main/java/org/apache/ranger/plugin/service/RangerAuthContext.java b9dff76e4 
  agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java a0808f91a 
  agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRolesUtil.java 81c278a43 
  agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyACLs.java bfe767e2c 
  security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyAdminImpl.java ac1d96193 
  security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyEngineCache.java a86f003f6 
  security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyEngineCacheForEngineOptions.java 151143abf 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 43c109dd3 
  security-admin/src/test/java/org/apache/ranger/biz/TestPolicyDb.java 51183223d 
  security-admin/src/test/java/org/apache/ranger/rest/TestServiceREST.java 75e93d928 


Diff: https://reviews.apache.org/r/71808/diff/1/


Testing
-------


Thanks,

Madhan Neethiraj


Re: Review Request 71808: RANGER-2652: refactor policy-engine #2 - renames, whitespace updates

Posted by Abhay Kulkarni <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71808/#review218774
-----------------------------------------------------------


Ship it!




Ship It!

- Abhay Kulkarni


On Nov. 22, 2019, 5:43 p.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71808/
> -----------------------------------------------------------
> 
> (Updated Nov. 22, 2019, 5:43 p.m.)
> 
> 
> Review request for ranger and Abhay Kulkarni.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> RangerPolicyEngineCache.java                 => RangerPolicyAdminCache.java
> RangerPolicyEngineCacheForEngineOptions.java => RangerPolicyAdminCacheForEngineOptions.java
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/PolicyEngine.java b24d37c5a 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessRequestProcessor.java a6836998d 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java afe6683fb 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java 28b441a44 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java 11c1eeb98 
>   agents-common/src/main/java/org/apache/ranger/plugin/service/RangerAuthContext.java b9dff76e4 
>   agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java a0808f91a 
>   agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRolesUtil.java 81c278a43 
>   agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyACLs.java bfe767e2c 
>   security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyAdminImpl.java ac1d96193 
>   security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyEngineCache.java a86f003f6 
>   security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyEngineCacheForEngineOptions.java 151143abf 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 43c109dd3 
>   security-admin/src/test/java/org/apache/ranger/biz/TestPolicyDb.java 51183223d 
>   security-admin/src/test/java/org/apache/ranger/rest/TestServiceREST.java 75e93d928 
> 
> 
> Diff: https://reviews.apache.org/r/71808/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>