Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR proposes reducing the default page size. It's certainly a change with some trade-offs, so feel free to reject it (or debate the extent of the reduction), but I will provide my argument below:
On lemm.ee, I have noticed that this small tweak has vastly improved the experience for our users. By reducing the page size in half, our page loads significantly faster, and server load has in general decreased slightly as well.
I will also share some actual stats. This was with
fetchLimit
set to 40:And this is the same test run with
fetchLimit
reduced to 20:Of course, the downside with this is that users need to load additional pages more often, but it seems like a small price to pay considering the performance benefits.