Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1078406 - [Docs][Developer] Expand the description of the pagination feature
[Docs][Developer] Expand the description of the pagination feature
Status: CLOSED CURRENTRELEASE
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: Documentation (Show other bugs)
3.3.0
x86_64 All
high Severity high
: ---
: 3.4.0
Assigned To: Andrew Burden
Lucy Bopf
: Documentation
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2014-03-19 12:50 EDT by Luca Villa
Modified: 2014-07-17 23:47 EDT (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2014-07-17 23:47:30 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Luca Villa 2014-03-19 12:50:17 EDT
Description of problem:

Looking at the provided developers guide pagination is only briefly mentioned.
There are a few elements that would deserve mentioning/expanding i.e.:

- the possible lack of read-consistency is not mentioned at all.
The REST-APIs are by design stateless. This means that there is no way to retain a state among different requests.
When you request to return a given result page from the e.g. the VMs' list, you are actually executing an independent request which is not related to other similar requests you may have already entered or you will enter subsequently.
So in this regard it's possible that a status change happens in the meantime while you request the following page and the result may be someway inconsistent across pages: you may miss existing entries or you may even have duplicated ones.
This is far from obvious from the documentation and it should be made very clear.

This request is about the need to cover pagination more extensively in the developer's guide.
Comment 2 Luca Villa 2014-04-03 12:02:22 EDT
As an addition to the above:

the use of the max= parameter to influence the number of results when the default number of 100 is insufficient should be also mentioned/duplicated on the pagination section as they seem to be strictly related.
A usage example would be even better.
Comment 3 Andrew Burden 2014-04-30 03:45:25 EDT
An Important admonition was added to 
Pagination [7720]

It reads:
" The REST APIs are stateless; it is not possible to retain a state between different requests since all requests are independent from each other. As a result, if a status change occurs between your requests, then the page results may be inconsistent.
For example, if you request a specific page from a list of VMs, and a status change occurs before you can request the next page, then your results may be missing entries or contain duplicated entries. "

It can be viewed here:
http://documentation-devel.engineering.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Virtualization/3.4/html-single/Technical_Guide/index.html#Pagination2

I have raised a separate bug for the 'max' parameter:
https://bugzilla.redhat.com/show_bug.cgi?id=1092880
Comment 4 Lucy Bopf 2014-07-17 23:47:30 EDT
Peer reviewed.

The 'Important' message from Comment 3 appears in the live docs.

VERIFIED.

Note You need to log in before you can comment on or make changes to this bug.