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

(Updated Sept. 1, 2016, 10:52 p.m.)


Review request for ranger and Madhan Neethiraj.


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


Repository: ranger


Description
-------

Support for variables in Ranger policy resource values can make it easy to manage policies; in many cases can help use a single policy to manage access permissions for a large number of resources.

The replacement algorithm for converting encoded policy-resource-specification into actual resource-identifier requires a little more work. Also, need to put in more detailed debugging information. Therefore, this is sent only to you for early review and feedback. Thanks!


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/conditionevaluator/RangerHiveResourcesAccessedTogetherCondition.java fc9842e 
  agents-common/src/main/java/org/apache/ranger/plugin/conditionevaluator/RangerHiveResourcesNotAccessedTogetherCondition.java 3b8e009 
  agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerServiceResourceMatcher.java cf7b8e7 
  agents-common/src/main/java/org/apache/ranger/plugin/contextenricher/RangerTagEnricher.java 5b60a53 
  agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceDef.java f6931b3 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngine.java a5e92da 
  agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java 6d3645f 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyItemEvaluator.java 514884f 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerCachedPolicyEvaluator.java 91a53d8 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java eb46353 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyItemEvaluator.java e2c715f 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerOptimizedPolicyEvaluator.java 00b24d1 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyEvaluator.java b60e06e 
  agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyItemEvaluator.java 80e46f5 
  agents-common/src/main/java/org/apache/ranger/plugin/policyresourcematcher/RangerDefaultPolicyResourceMatcher.java 8bde807 
  agents-common/src/main/java/org/apache/ranger/plugin/policyresourcematcher/RangerPolicyResourceMatcher.java a8810e5 
  agents-common/src/main/java/org/apache/ranger/plugin/resourcematcher/RangerAbstractResourceMatcher.java 574f2eb 
  agents-common/src/main/java/org/apache/ranger/plugin/resourcematcher/RangerDefaultResourceMatcher.java 0a11be0 
  agents-common/src/main/java/org/apache/ranger/plugin/resourcematcher/RangerPathResourceMatcher.java d508f3f 
  agents-common/src/main/java/org/apache/ranger/plugin/resourcematcher/RangerResourceMatcher.java 8f1cebe 
  agents-common/src/main/java/org/apache/ranger/plugin/resourcematcher/ResourceMatcher.java 39eb339 
  agents-common/src/main/java/org/apache/ranger/plugin/util/RangerAccessRequestUtil.java 0ce3721 
  agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRequestedResources.java 0f10deb 
  agents-common/src/main/java/org/apache/ranger/plugin/util/RangerResourceTrie.java 2079487 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ServiceDefUtil.java 0a2b451 
  agents-common/src/main/java/org/apache/ranger/plugin/util/StringTokenReplacer.java PRE-CREATION 
  agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/RangerAbstractResourceMatcherTest.java 48bc6ee 
  agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/RangerDefaultResourceMatcherTest.java d2fb62c 
  agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/RangerPathResourceMatcherTest.java c9d207f 
  agents-common/src/test/java/org/apache/ranger/plugin/resourcematcher/TestResourceMatcher.java 9b870d4 
  agents-common/src/test/resources/policyengine/test_policyengine_hdfs_resourcespec.json da0a629 
  agents-common/src/test/resources/resourcematcher/test_resourcematcher_default.json 918c30f 
  agents-common/src/test/resources/resourcematcher/test_resourcematcher_dynamic.json PRE-CREATION 
  agents-common/src/test/resources/resourcematcher/test_resourcematcher_path.json 25b0eb7 
  agents-common/src/test/resources/resourcematcher/test_resourcematcher_wildcards_as_delimiters.json PRE-CREATION 
  security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java 5e8c540 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 90b146b 

Diff: https://reviews.apache.org/r/51232/diff/


Testing
-------

Ran all unit tests successfully.


Thanks,

Abhay Kulkarni