-----------------------------------------------------------
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