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

next_batch token in the last page #82

Open
anteverse opened this issue Feb 24, 2022 · 1 comment
Open

next_batch token in the last page #82

anteverse opened this issue Feb 24, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@anteverse
Copy link

Bonjour,

La stratégie actuelle permet d'obtenir 10k résultats pour une query donnée. Et le franchissement de cette limite retourne un statut 416.
Lorsqu'on requête l'avant-dernier batch, le champ next_batch dans la réponse devient null, ce qui a du sens car la prochaine requête donnera un statut 416.

Serait possible de continuer à fournir un next_batch pour lequel la query a changé, et plus particulièrement les filtres sur les dates lors d'un appel à /export par exemple.

Un exemple :

Plusieurs notes à ce sujet :

  • j'imagine que la limite de 10k résultats n'est pas un rate-limiter mais plutôt une façon de limiter la complexité de la pagination sur la base source ? De plus on a déjà un rate-limit sur le compte Piste.
  • ce changement de query est possible par le client déjà, peut-être qu'il pourrait être géré côté serveur plutôt ?
  • lors du switch de query, il y a potentiellement un overlap sur les dates, ce pourrait être géré avec des timestamps.

À l'inverse, si c'est vraiment une mesure de rate-limit, à l'heure actuelle rien n'empêche le client de switcher tout seul et donc le rate-limit est contourné et ce n'est pas le comportement voulu.

S'il y a une meilleure stratégie à adopter côté client, n'hésitez pas à me le signaler. Disponible pour contribuer et tester.

@SebCourvoisier
Copy link
Collaborator

Bonjour,
Problème analogue à #88 (limitation intrinsèque), contournement en cours d'étude.

@SebCourvoisier SebCourvoisier self-assigned this Jul 3, 2022
@SebCourvoisier SebCourvoisier added the enhancement New feature or request label Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants