Remove fetching of Realm notifications from /realms/slug/{realm_name}
endpoint
#162
Open
3 of 7 tasks
Labels
📦 boba-backend
An issue that involves the boba-backend repository
⏩ Implementation
A feature that's currently being implemented
🚢 Large
🤸♂️ Teamwork welcome
An issue made of many small parts that can be tackled by multiple people independently
Problem
Fetching of Realm notifications is really slow and needs to be redesigned. Luckily, we already did some work splitting the "realm data fetching" endpoint (
/realms/slug/{realm_name}
) and the "realm notifications" endpoint (/realms/{realm_id}/notifications
). However, the data fetching endpoint still uses the query that also fetches notifications, which is less than ideal.Let's fix that!
The Players
getRealmBoards
is our entry point into the infamous "white whale query", the worst performing boba query that we have declared unfixable.Step by Step
getRealmBoards
query as data comes out of the database/realms/slug/{realm_name}
endpoint/realms/{realm_id}/notifications
endpoint/realms/slug/{realm_name}
endpoint/realms/slug/{realm_name}
endpoint to use that queryStretch
getRealmBoards
query is used in other parts of the codebase that don't need it.The Future
While this gets fixed we'll be working on improving the performance of the notifications query by leveraging Redis.
The text was updated successfully, but these errors were encountered: