Skip to content
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

Remove old page-based pagination from /ras/runs API in favour of cursor-based pagination #1978

Open
2 tasks
eamansour opened this issue Aug 22, 2024 · 0 comments

Comments

@eamansour
Copy link
Member

Story

As a Galasa maintainer, I want to remove the existing slow page-based pagination from the /ras/runs API and replace it with the new cursor-based pagination, so that REST API users benefit from the increased performance of cursor-based pagination.

Background

This story assumes 0.37.0 has been released and the development version is now at 0.38.0.

As part of #1921, cursor-based pagination was added to the /ras/runs endpoint that could be enabled/disabled using the includeCursor query parameter. This was added because the existing page-based pagination was much slower due to it retrieving all the runs matching a given query and discarding the runs that aren't in the page requested, every time a page was requested.

A PR was opened to replace the old page-based pagination with the new approach galasa-dev/framework#627 - it wasn't merged in to avoid causing issues with backwards compatibility. With the release of 0.37.0, there has been enough time for clients to migrate to using cursor-based pagination, so the old pagination can be removed.

Tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 5 Backlog
Development

No branches or pull requests

1 participant