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

List endpoints could benefit from MGET access for Redis(tm) #1602

Open
bctcvai opened this issue Apr 2, 2024 · 0 comments
Open

List endpoints could benefit from MGET access for Redis(tm) #1602

bctcvai opened this issue Apr 2, 2024 · 0 comments
Milestone

Comments

@bctcvai
Copy link
Member

bctcvai commented Apr 2, 2024

Where we check cache entries on list endpoints MGET should increase throughput when accessing large lists. It should also be observed not to execute SQL statements in a loop. An example here is rest/Projects which is guilty of both.

I did not check if other endpoints already utilize MGET, its unlikely they have the same loop-based thumbnail access.

@bctcvai bctcvai added this to the 1.4.0 milestone Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant