Re: Review Request 37613: RANGER-611 : Update policy listing page with a new column 'Policy Type'

[Madhan Neethiraj <ma...@apache.org>]

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

Ship it!


Ship It!

- Madhan Neethiraj


On Aug. 19, 2015, 1:59 p.m., Gautam Borad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37613/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2015, 1:59 p.m.)
> 
> 
> Review request for ranger, Alok Lal, Don Bosco Durai, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-611
>     https://issues.apache.org/jira/browse/RANGER-611
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> With enhancements in RANGER-606, Ranger support 2 new types of policies: DENY, EXCLUSIVE_ALLOW. This is in addition to existing policies, which are of type ALLOW.
> 
> Policy listing page should be updated with a new column 'Policy Type', perhaps right next to 'Policy Name'.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/rest/AssetREST.java b931f7b 
>   security-admin/src/main/java/org/apache/ranger/service/XAccessAuditService.java 20079fb 
>   security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js e306ec4 
>   security-admin/src/main/webapp/scripts/views/reports/PlugableServiceDiffDetail.js ceb8898 
>   security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js b1ed454 
> 
> Diff: https://reviews.apache.org/r/37613/diff/
> 
> 
> Testing
> -------
> 
> Tested on a local Ranger setup for all policy types in policy form
> 
> 
> Thanks,
> 
> Gautam Borad
> 
>