perf: fix caching #8660

Merged

Johann150 merged 1 commits from perf-cache into develop

2022-05-14 04:28:28 +00:00

Johann150 commented

2022-05-13 18:17:09 +00:00

(Migrated from github.com)

What

Fixes the cache implementation to actually store the result when fetching a value for a cache miss.

Why

The cache implementation did previously not store the results of the computation and was thus not a cache at all. This can cause a significant number of database queries each time someone with a large number of followers does something that causes an activity to be federated.

# What Fixes the cache implementation to actually store the result when fetching a value for a cache miss. # Why The cache implementation did previously not store the results of the computation and was thus not a cache at all. This can cause a significant number of database queries each time someone with a large number of followers does something that causes an activity to be federated.

syuilo commented

2022-05-14 04:28:39 +00:00

(Migrated from github.com)

Good catch

Good catch

Sign in to join this conversation.

No reviewers

No Label

1️⃣ Good First Issue

❓needs more investigation

Posts/Timelines

‼️ wrong locales

♻️Duplicate

🔥high priority

🙏low priority

🐢Performance

🤖Service Worker

👩‍💻AiScript

🏳️needs help

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: firefish/firefish#8660

There is no content yet.