----------------------------------------------------------- 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 > >