You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 29, 2021. It is now read-only.
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.
These values are broken in a number of places including the SchemaIdsRequest and any shim request.
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.
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).
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
SchemaIdsRequest
and any shim request.MultiValueResult
andMultiValueResultAggregator
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.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).
The text was updated successfully, but these errors were encountered: