Skip to content
This repository has been archived by the owner on Oct 18, 2024. It is now read-only.

Add DynamoDB-backed cache for select endpoints #98

Closed
ecxyzzy opened this issue Sep 9, 2023 · 1 comment
Closed

Add DynamoDB-backed cache for select endpoints #98

ecxyzzy opened this issue Sep 9, 2023 · 1 comment
Assignees
Labels
area: api Related to the API in general story point: 3 Change to endpoint or response type: feature request Brand new functionality, features, pages, workflows, endpoints, etc

Comments

@ecxyzzy
Copy link
Member

ecxyzzy commented Sep 9, 2023

@libs/ddb be like:
image


Biggest offenders right now are probably grades and WebSoc, and those will definitely be cached. Feel free to suggest any other endpoints that might need caching below.

Blocked by #96.

@ecxyzzy ecxyzzy added story point: 3 Change to endpoint or response type: feature request Brand new functionality, features, pages, workflows, endpoints, etc area: api Related to the API in general labels Sep 9, 2023
@ecxyzzy ecxyzzy self-assigned this Sep 9, 2023
@ecxyzzy ecxyzzy moved this from Backlog to In Progress in PeterPortal API :: Next Feb 5, 2024
@ecxyzzy
Copy link
Member Author

ecxyzzy commented Feb 6, 2024

After further testing it appears that adding the cache generates a negligible performance boost. Maybe we just got good in five months, so going to put this in the icebox for now.

@ecxyzzy ecxyzzy closed this as not planned Won't fix, can't repro, duplicate, stale Feb 6, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in PeterPortal API :: Next Feb 6, 2024
@ecxyzzy ecxyzzy moved this from Done to Icebox in PeterPortal API :: Next Feb 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: api Related to the API in general story point: 3 Change to endpoint or response type: feature request Brand new functionality, features, pages, workflows, endpoints, etc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant