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

Optimize Drafts retrieval #295

Merged
merged 4 commits into from May 29, 2020
Merged

Optimize Drafts retrieval #295

merged 4 commits into from May 29, 2020

Conversation

@thebaer
Copy link
Member

thebaer commented Apr 9, 2020

This adds a database index to speed up retrieval of Drafts.

Most small instances won't notice any difference from this change, but it'll ensure they stay fast as they grow. It was driven by the performance issues we've encountered on Write.as as we've approached 800,000 posts on the platform. Response times for our "Drafts" page (labelled as "Posts") dropped from several seconds to under half a second with this change.

This is a draft, since it's untested with SQLite. Ready for review now.

This adds a database index to speed up retrieval of Drafts.

It is untested with SQLite.
@thebaer thebaer added this to the 0.13 milestone Apr 9, 2020
@thebaer thebaer marked this pull request as ready for review May 6, 2020
@thebaer
Copy link
Member Author

thebaer commented May 29, 2020

Merging now.

@thebaer thebaer merged commit 2aa154d into develop May 29, 2020
@thebaer thebaer deleted the optimize-drafts-fetch branch May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.