Re: Review Request 56069: RANGER-1336 : Audit based policy that has no policy item are not exported in CSV file

[Velmurugan Periasamy <vp...@hortonworks.com>]

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


Ship it!




Ship It!

- Velmurugan Periasamy


On Jan. 31, 2017, 4:58 a.m., Gautam Borad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56069/
> -----------------------------------------------------------
> 
> (Updated Jan. 31, 2017, 4:58 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Don Bosco Durai, Madhan Neethiraj, Pradeep Agrawal, Selvamohan Neethiraj, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1336
>     https://issues.apache.org/jira/browse/RANGER-1336
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Audit policies that did not have any policy items were not exported in CSV file. This patch fixes that.
> 
> 
> Diffs
> -----
> 
>   kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java 021685c 
>   kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMetricUtil.java 22fb03c 
>   security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 9a4e571 
> 
> Diff: https://reviews.apache.org/r/56069/diff/
> 
> 
> Testing
> -------
> 
> Applied patch on a locally setup Ranger and did the following steps:
> 
> 1) Create a yarn policy without any policy item
> 2) Go to report page
> 3) Export as csv file
> 
> Checked the csv file and the file had audit based yarn policy.
> 
> 
> Thanks,
> 
> Gautam Borad
> 
>