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

Redis caching for pagination for posts and comments: only for dynamic fields. #77

Open
mattrltrent opened this issue Jul 1, 2023 · 0 comments
Labels
enhancement Enhancing something that exists

Comments

@mattrltrent
Copy link
Member

Instead of using Redis cache for both paginating by best (vote field?) or newest (created_at field?), we should only use the cache for dynamic fields, and have a different endpoint with a cursor for static sort fields.

This will drastically increase efficiency, as less things will be redundantly stored in Redis.

@mattrltrent mattrltrent added the enhancement Enhancing something that exists label Jul 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancing something that exists
Projects
None yet
Development

No branches or pull requests

1 participant