Skip to content
This repository has been archived by the owner on Apr 29, 2021. It is now read-only.

Prev/Next and Count/Total-Count headers for ListResults #37

Open
mschulkind opened this issue May 27, 2014 · 0 comments
Open

Prev/Next and Count/Total-Count headers for ListResults #37

mschulkind opened this issue May 27, 2014 · 0 comments

Comments

@mschulkind
Copy link

Any DSU request that returns a list of data support the num_to_skip and num_to_return parameters. They also return a number of headers describing the omitted data and links to the next/prev pages.

There are currently a number of issues with this.

  1. These values are broken in a number of places including the SchemaIdsRequest and any shim request.
  2. The MultiValueResult and MultiValueResultAggregator and related code that are supposed to handle deriving these values is brittle, awkward, and hard to understand, leading to breakages such as in item 1.
  3. The shim interface does not support passing back the information required to correctly calculate these values, and even if it did, it's not always possible for a shim to calculate the required data.

My recommendation is to just remove all of this and let the user page through the data and stop when they encounter then end (empty request).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant