Alon Bar-Lev has posted comments on this change. Change subject: engine : [AAA] Get users by domain returns partial list of users ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/30544/1/backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java File backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java: Line 25 Line 26 Line 27 Line 28 Line 29 > api also accumulates all in memory so we need to limit it for api as well. yair, btw, this also effect our sync... if we plan to have more than 1000 users in database we should move paging to sync as well. -- To view, visit http://gerrit.ovirt.org/30544 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie4c121b0edd6966c518b46aa8e98c24be6cb74c8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
