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

storage-http: pagination with next url extracted from the page? #2828

Open
matteoredaelli opened this issue Aug 25, 2023 · 2 comments
Open
Labels
enhancement PRs that add a new functionality to Drill

Comments

@matteoredaelli
Copy link

Hello

Qliksaas apis use pagination without offset params. They provide the "next" url in the response page

How could I tell drill to use "links.next.href" value for retreiving further records?

https://qlik.dev/apis/pagination-sorting-filtering

{
  "data": ["list", "of", "items"],
  "links": {
    "prev": { "href": null },
    "next": { "href": "https://example.org/api/v1/items?page=<next-cursor>" },
  }
}
@cgivre cgivre added the enhancement PRs that add a new functionality to Drill label Aug 25, 2023
@cgivre
Copy link
Contributor

cgivre commented Aug 25, 2023

Take a look here: https://github.com/apache/drill/blob/master/contrib/storage-http/Pagination.md. Drill might already do what you need to do.

@matteoredaelli
Copy link
Author

matteoredaelli commented Aug 26, 2023

Thanks, you are right. It is in the section "Index / KeySet Pagination" : I need to add "nextPageParam" parameter but it is not clear which "method" I should set for "Index / KeySet Pagination" (it is indicated for the others options). I think "INDEX" as I see in teh code

  public enum PaginatorMethod {
    OFFSET,
    PAGE,
    INDEX,
    HEADER_INDEX
  }

Thanks
Matteo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement PRs that add a new functionality to Drill
Projects
None yet
Development

No branches or pull requests

2 participants