Keith Robertson has posted comments on this change.

Change subject: Use the max=-1 option return unlimited results when calling the 
oVirt API.
......................................................................


Patch Set 2: Code-Review+1

I think this is a good first step but the optimal solution is to use 
pagination.  This solution will break down if (and it is possible) if the 
Engine is incapable of returning all results.  In those instances you need 
pagination to page through the results.

-- 
To view, visit http://gerrit.ovirt.org/19759
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0c9f61938df1626b1d81e4c0320b5bca10614f34
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: master
Gerrit-Owner: Lee Yarwood <[email protected]>
Gerrit-Reviewer: Keith Robertson <[email protected]>
Gerrit-Reviewer: Kiril Nesenko <[email protected]>
Gerrit-Reviewer: Lee Yarwood <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to