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

(Updated Dec. 7, 2020, 6:11 p.m.)


Review request for ranger, Madhan Neethiraj, Ramesh Mani, Sailaja Polavarapu, 
and Velmurugan Periasamy.


Changes
-------

Provide Apache JIRA details


Summary (updated)
-----------------

RANGER-3107: Optimize memory requirements for storing Tag/Policy/Zone trie - 
Trade-off processing time for memory


Bugs: RANGER-3107
    https://issues.apache.org/jira/browse/RANGER-3107


Repository: ranger


Description (updated)
-------

Trie structure is devised to index tagged entities and policies for a fast 
look-up. However, it may be take a lot of memory. This is a result of a design 
decision to optimize processing (mainly searching) time at the cost of extra 
memory.

It is desirable to find a more optimal trade-off between memory needs and 
processing requirements, especially for supporting a large number or policies 
and tagged entities.


Diffs (updated)
-----

  
agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerTagEnricher.java
 d1afd6f58 
  
agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerSecurityZoneValidator.java
 2db2f222f 
  
agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceDefHelper.java
 bbcf82f89 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/PolicyEngine.java
 27423125c 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
 ffbd90878 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceTrie.java
 44285032a 
  
agents-common/src/test/java/org/apache/ranger/plugin/contextenricher/TestTagEnricher.java
 83b39eda0 
  
agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyEngineComparison.java
 e2049832f 


Diff: https://reviews.apache.org/r/73063/diff/4/

Changes: https://reviews.apache.org/r/73063/diff/3-4/


Testing
-------

Ran all unit tests successfully.


Thanks,

Abhay Kulkarni

Reply via email to