-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add pagination to the REST API v3 #2060
Labels
feature
New feature
priority: high
Critical to the health of the project
under discussion
Requires consideration before a decision is made whether/how to implement
Comments
Zejnilovic
added
feature
New feature
priority: undecided
Undecided priority to be assigned after discussion
under discussion
Requires consideration before a decision is made whether/how to implement
labels
May 4, 2022
dk1844
added
priority: high
Critical to the health of the project
and removed
priority: undecided
Undecided priority to be assigned after discussion
labels
Jun 7, 2022
dk1844
added a commit
that referenced
this issue
Jun 9, 2022
dk1844
added a commit
that referenced
this issue
Jun 9, 2022
…et`: Int, `limit`: Int, `truncated`: Boolean) + IT coverage - test coverage uses `TestPaginated` (array-based) to correctly asses wrapped types while deserialization. Custom matcher added to check conforming TestPaginated objects
dk1844
added a commit
that referenced
this issue
Jun 9, 2022
dk1844
added a commit
that referenced
this issue
Jun 9, 2022
dk1844
added a commit
that referenced
this issue
Jun 10, 2022
dk1844
added a commit
that referenced
this issue
Jun 10, 2022
dk1844
added a commit
that referenced
this issue
Jul 18, 2022
dk1844
added a commit
that referenced
this issue
Jul 18, 2022
* #2060 - pagination for Datasets, Schema, PDs, MTs, RunSummary (/runs, /runs/{datasetName},/runs/{datasetName}/{datasetVersion}) via `Paginated` wrapper (features `offset`: Int, `limit`: Int, `truncated`: Boolean) - common pagination functionality for controllers moved to `ControllerPagination` object - pagination params sanitization -> invalid value (non-int), incorrect value (negative) - IT coverage - test coverage uses `TestPaginated` (array-based) to correctly assesses wrapped types while deserialization. Custom matcher added to check conforming TestPaginated objects Co-authored-by: David Benedeki <benedeki@volny.cz> Co-authored-by: Saša Zejnilović <zejnils@gmail.com>
Release notesSome potentionally large container-listing endpoints are now paginated, with |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature
New feature
priority: high
Critical to the health of the project
under discussion
Requires consideration before a decision is made whether/how to implement
Background
There are calls which pull a lot of data.
/datasets
and/runs
. These calls should be paginated, maybe others as well.Feature
Add pagination to the REST API v3
The text was updated successfully, but these errors were encountered: