> On Aug. 8, 2023, 12:37 p.m., Ramachandran Krishnan wrote: > > security-admin/src/main/java/org/apache/ranger/db/XXUserDao.java > > Lines 130 (patched) > > <https://reviews.apache.org/r/74528/diff/6/?file=2278066#file2278066line130> > > > > Is it possible to move this native query into jpa_named_queries.xml and > > passing the binding parameter > > Prashant Satam wrote: > can do that but will require 3 seperate JPA queries as UNION is not > supported in JPA query
Consider using a VIEW to get around JPA limitation on use of UNION. - Madhan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74528/#review225634 ----------------------------------------------------------- On Aug. 7, 2023, 9:12 a.m., Prashant Satam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74528/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2023, 9:12 a.m.) > > > Review request for ranger, Madhan Neethiraj and Subhrat Chaudhary. > > > Bugs: RANGER-4325 > https://issues.apache.org/jira/browse/RANGER-4325 > > > Repository: ranger > > > Description > ------- > > We get users,groups,roles all we get in response from one API > (/xusers/users/groups/roles) we can also pass Query Params like > name,isVisible Also we can PartialSearch Name for users,groups,roles > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 23650c8d5 > security-admin/src/main/java/org/apache/ranger/db/XXUserDao.java 283d84fe1 > security-admin/src/main/java/org/apache/ranger/rest/XUserREST.java > 9a2253a3d > > > Diff: https://reviews.apache.org/r/74528/diff/6/ > > > Testing > ------- > > We get users,groups,roles combined in response from API > (/xusers/users/groups/roles) we can pass query params like name(PartialSearch > Available),isVisible to filter the users,groups,roles we get in response > > > Thanks, > > Prashant Satam > >
