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

(Updated March 9, 2015, 1:43 p.m.)


Review request for ranger.


Changes
-------

Addressed review comments:
- id is ignored for create policy/service
- update can update name policy/service


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


Repository: ranger


Description
-------

Added validation for policy create/update/delete operations. Moved action out 
of ctor to validate call.


Diffs (updated)
-----

  agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java 
ab8384c 
  
security-admin/src/main/java/org/apache/ranger/rest/RangerPolicyValidator.java 
PRE-CREATION 
  
security-admin/src/main/java/org/apache/ranger/rest/RangerServiceValidator.java 
08184c7 
  security-admin/src/main/java/org/apache/ranger/rest/RangerValidator.java 
3f25266 
  
security-admin/src/test/java/org/apache/ranger/rest/TestRangerPolicyValidator.java
 PRE-CREATION 
  
security-admin/src/test/java/org/apache/ranger/rest/TestRangerServiceValidator.java
 3bbb123 

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


Testing
-------

Additional junits added.  Patch allies to latest trunk.


Thanks,

Alok Lal

Reply via email to