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


Ship it!




Ship It!

- Monika Kachhadiya


On Jan. 9, 2024, 10:58 p.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/74826/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2024, 10:58 p.m.)
> 
> 
> Review request for ranger, Anand Nadar, Ankita Sinha, Abhay Kulkarni, Mehul 
> Parikh, Monika Kachhadiya, Mugdha Varadkar, Ramesh Mani, Sailaja Polavarapu, 
> Subhrat Chaudhary, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-4642
>     https://issues.apache.org/jira/browse/RANGER-4642
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> - updated RangerSecurityZoneServiceService.preDelete() and postUpdate() to 
> remove associated GDS objects
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/plugin/store/GdsStore.java 
> afbce78a1 
>   security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java 
> 701165f9d 
>   security-admin/src/main/java/org/apache/ranger/db/XXGdsDataShareDao.java 
> 2845a1d04 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 
> e7e0abbc8 
>   
> security-admin/src/main/java/org/apache/ranger/service/RangerSecurityZoneServiceService.java
>  940bd0bc6 
>   
> security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java
>  74a3caa13 
>   security-admin/src/main/resources/META-INF/jpa_named_queries.xml ae6788bac 
> 
> 
> Diff: https://reviews.apache.org/r/74826/diff/1/
> 
> 
> Testing
> -------
> 
> - verified that deletion of a security-zone deletes all data-shares 
> associated with the security-zone
> - verified that removal of services from a security-zone deletes all 
> data-shares of the services in the security-zone
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>

Reply via email to