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

change pagination to use Elasticsearch search_after instead of 'page' #228

Merged
merged 18 commits into from
Mar 11, 2022

Conversation

philvarner
Copy link
Collaborator

Related Issue(s):

Proposed Changes:

  1. use ES search_after instead of page for next links
  2. Removes prev links (not possible to do easily with search_after)

PR Checklist:

  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@philvarner philvarner changed the title Pv/change pagination to search after change pagination to use Elasticsearch search_after instead of 'page' Mar 9, 2022
@philvarner philvarner marked this pull request as ready for review March 11, 2022 17:50
@philvarner philvarner merged commit dcb50d2 into main Mar 11, 2022
@philvarner philvarner deleted the pv/change-pagination-to-search-after branch March 11, 2022 18:51
@tschaub
Copy link

tschaub commented May 24, 2022

I'm seeing the deep paging issue on https://landsatlook.usgs.gov/stac-server (e.g. https://landsatlook.usgs.gov/stac-server/collections/landsat-c2l3-ba/items?collections[]=%5B%22landsat-c2l3-ba%22%5D&page=41&limit=250).

I know this is the wrong place to ask, but does anybody know who maintains the LandsatLook STAC API? It would be great if it could be updated to stac-server@0.4.0. I'll give https://answers.usgs.gov/ a try too.

@vincentschut
Copy link

We're running into the same 10000-limit issue with the landsatlook stac server. @tschaub did you ever try to contact them about this and got a reply?

@tschaub
Copy link

tschaub commented Feb 2, 2023

Response from custerv@usgs.gov on 2022/05/31:

The USGS is on a one year rotating schedule for updating our STAC versioning. We have an update coming shortly, but we cannot keep up to date on all version releases as we have a very involved/ in depth check out process to vet any new updates.

Thanks for your understanding.
Regards,

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

Successfully merging this pull request may close these issues.

4 participants