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

Ik wil het aantal resultaten limiteren... #2490

Open
sanderdekroon opened this issue Nov 11, 2024 · 3 comments
Open

Ik wil het aantal resultaten limiteren... #2490

sanderdekroon opened this issue Nov 11, 2024 · 3 comments

Comments

@sanderdekroon
Copy link

...zodat ik een voorspelbare, gepagineerde weergave kan opbouwen.

Context

In OpenZaak wordt gebruik gemaakt van de pageSize parameter, waarmee het aantal resultaten beperkt kan worden tot het opgegeven aantal. Deze parameter ontbreekt echter in de standaard en zou naar mijn mening toegevoegd moeten worden.

Ik zie nu dat sommige applicaties alle resources teruggeven en dat is alles behalve ideaal. De requests duren langer en het verwerken van alle data kost ook meer tijd. Het toevoegen van de pageSize parameter maakt het voor de consumerende applicatie mogelijk om hier zelf controle over te houden. Wel zo fijn lijkt mij :)

CC @EduardWitteveen

@terborg
Copy link
Collaborator

terborg commented Feb 20, 2025

Je primaire concern lees ik als de tijdsduur van requests? Dan is het wellicht een idee om naast het limiteren van de result set size ook naar een concept als Deadline kijken, waarin je als client zou kunnen aangeven hoe lang je bereid zou zijn te wachten (evt op een niet van tevoren bekend aantal results).

@sanderdekroon
Copy link
Author

sanderdekroon commented Feb 25, 2025

Ja deels tijdsduur, maar met name voorspelbaarheid. Deadline is een leuk concept, maar ik zie meer heil in een pageSize parameter. Daarmee kan daadwerkelijk een gepagineerde weergave opbouwen. Een deadline parameter helpt mij in die zin niet.

@HenriKorver
Copy link
Collaborator

Dit issue is gerelateerd aan #2284. Mijn suggestie is om pageSize mee te nemen met de countExact parameter.

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

No branches or pull requests

3 participants