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

Taille limite de l'export trop petite, ou mises à jour trop volumineuses #88

Open
atomobianco opened this issue Apr 15, 2022 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@atomobianco
Copy link

atomobianco commented Apr 15, 2022

Des problèmes dans la conception des limites de l'API /export, ou bien je n'utilise pas les bons paramètres.
L'export est limité à 10 k decisions. Chaque jour je dois récupérer les décisions nouvelles et les décisions qui ont été mises à jour. J'utilise donc le paramètre date_type=update sur une fenêtre d'un jour. Mais le 14 Avril 2022, 21901 décisions ont été mises à jour. Donc pas moyen de les récupérer en entier ?

@anteverse
Copy link

+1 @atomobianco

Le problème est récurrent ces-derniers jours.

@matheyal
Copy link

matheyal commented Apr 21, 2022

Le soucis provient d'une limitation de la pagination "par défaut" de l'API search d'Elasticsearch.

Extrait de la doc Elasticsearch:

By default, you cannot use from and size to page through more than 10,000 hits. This limit is a safeguard set by the index.max_result_window index setting. If you need to page through more than 10,000 hits, use the search_after parameter instead.

Il faudrait donc modifier la manière dont la query d'export est construite dans la méthode buildQuery() de manière à utiliser les "point-in-time" et le paramètre search_after

@cjolif
Copy link

cjolif commented Apr 22, 2022

Voir aussi #91

@SebCourvoisier
Copy link
Collaborator

Le problème survient lors des phases d'évolutions fonctionnelles (comme l'intégration des décisions de cours d'appel) nécessitant la réindexation du stock. Des limitations doivent être mises en place en amont, en complément d'une solution analogue à celle qui est évoquée ci-dessus : #88 (comment).

@SebCourvoisier SebCourvoisier self-assigned this Jul 3, 2022
@SebCourvoisier SebCourvoisier added the bug Something isn't working label Jul 5, 2022
SebCourvoisier added a commit that referenced this issue Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants