diff options
author | ThibG <thib@sitedethib.com> | 2020-04-05 12:51:22 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-05 12:51:22 +0200 |
commit | 89e28c76744dc888e7c5f85aef305452681fd6be (patch) | |
tree | a8912717fa31375c5898786e370ed022e3c43c5c /app/lib | |
parent | a889756dd5f72a1423a613d8ce141f5347da48bc (diff) |
Fix PostgreSQL load when linking in announcements (#13250)
* Fix PostgreSQL load when linking in announcements Fixes #13245 by caching status lookups Since statuses are supposed to be known already and we only need their URLs and a few other things, caching them should be fine. Since it's only used by announcements so far, there won't be much statuses to cache. * Perform status lookup when saving announcements, not when rendering them * Change EntityCache#status to fetch URLs instead of looking into the database * Move announcement link lookup to publishing worker * Address issues pointed out during review
Diffstat (limited to 'app/lib')
-rw-r--r-- | app/lib/entity_cache.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/app/lib/entity_cache.rb b/app/lib/entity_cache.rb index 35a3773d2..afdbd70f2 100644 --- a/app/lib/entity_cache.rb +++ b/app/lib/entity_cache.rb @@ -7,6 +7,10 @@ class EntityCache MAX_EXPIRATION = 7.days.freeze + def status(url) + Rails.cache.fetch(to_key(:status, url), expires_in: MAX_EXPIRATION) { FetchRemoteStatusService.new.call(url) } + end + def mention(username, domain) Rails.cache.fetch(to_key(:mention, username, domain), expires_in: MAX_EXPIRATION) { Account.select(:id, :username, :domain, :url).find_remote(username, domain) } end |