----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74767/#review226028 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On Nov. 29, 2023, 9:39 a.m., Prashant Satam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74767/ > ----------------------------------------------------------- > > (Updated Nov. 29, 2023, 9:39 a.m.) > > > Review request for ranger, Anand Nadar, Ankita Sinha, Madhan Neethiraj, > Monika Kachhadiya, Subhrat Chaudhary, and Vanita Ubale. > > > Bugs: RANGER-4542 > https://issues.apache.org/jira/browse/RANGER-4542 > > > Repository: ranger > > > Description > ------- > > For GET API service/zones/summary we need to add support for query param > search by zoneNamePartial , and sort by zoneName, createTime (Both ASC and > DESC order) > When we pass just sortTpye/sortBy params single or together we dont get > response accordingly > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractPredicateUtil.java > 07c561506 > > agents-common/src/main/java/org/apache/ranger/plugin/store/SecurityZonePredicateUtil.java > 9164091ae > security-admin/src/main/java/org/apache/ranger/biz/SecurityZoneDBStore.java > 6c6ddc49f > > > Diff: https://reviews.apache.org/r/74767/diff/1/ > > > Testing > ------- > > Steps to Test > 1)Create multiple security-zones > 2)Use /service/zones/summary?sortBy=zoneName&sortType=DESC only using > (sortTpye/sortBy) we get response according to this params > 3)Use just /service/zones/summary?sortType=DESC we get response according to > this param > 4)Create 2 security-zones with similar names i.e Test-zone-1 and Test-zone-11 > 5)Use /service/zones/summary?zoneNamePartial=Test-zone-1 we get 2 > security-zones in response > > > Thanks, > > Prashant Satam > >
