----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72939/#review224846 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/db/XXGroupUserDao.java Lines 204 (patched) <https://reviews.apache.org/r/72939/#comment313725> Per description in https://issues.apache.org/jira/browse/RANGER-3929, most databases have a limit of 1000 entries for the list used in "IN" clause. Please review the impact of such limit here. security-admin/src/main/java/org/apache/ranger/view/VXUser.java Lines 329 (patched) <https://reviews.apache.org/r/72939/#comment313724> hashCode() - is not defined in base classes of VXUser. This call here would result in executing Object.hashCode(), which may not be the intent here. Please review and update. - Madhan Neethiraj On Oct. 29, 2022, 10:53 a.m., Bill Ricky wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72939/ > ----------------------------------------------------------- > > (Updated Oct. 29, 2022, 10:53 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-3027 > https://issues.apache.org/jira/browse/RANGER-3027 > > > Repository: ranger > > > Description > ------- > > RANGER-3027: Improve response time for GET API service/xusers/users > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/db/XXGroupUserDao.java > 76cdd93f5 > security-admin/src/main/java/org/apache/ranger/db/XXPortalUserDao.java > 1787eeae6 > security-admin/src/main/java/org/apache/ranger/entity/XXUser.java 99bea26f3 > security-admin/src/main/java/org/apache/ranger/service/XUserService.java > d7c011ca3 > > security-admin/src/main/java/org/apache/ranger/service/XUserServiceBase.java > 25fec3dd4 > security-admin/src/main/java/org/apache/ranger/view/VXUser.java 59375c42a > security-admin/src/main/resources/META-INF/jpa_named_queries.xml 85c8b6213 > > > Diff: https://reviews.apache.org/r/72939/diff/5/ > > > Testing > ------- > > done > > > File Attachments > ---------------- > > 0001-RANGER-3027-Improve-response-time-for-GET-API-servic.patch > > https://reviews.apache.org/media/uploaded/files/2022/10/29/88fd4c1a-32cf-4c2a-9b64-0e64471d39bf__0001-RANGER-3027-Improve-response-time-for-GET-API-servic.patch > > > Thanks, > > Bill Ricky > >
