diff options
| author | 2025-10-15 13:32:02 +0200 | |
|---|---|---|
| committer | 2025-10-17 15:33:35 +0200 | |
| commit | 2bdff66f0a12a16684e5d25bcace551446ec1c78 (patch) | |
| tree | 4a667a71f97d6d2d52d5b3dae1e56d74192219b3 /internal/cache | |
| parent | [chore/performance] Use CTE for list select statuses query (#4501) (diff) | |
| download | gotosocial-2bdff66f0a12a16684e5d25bcace551446ec1c78.tar.xz | |
[performance] cache account IDs in home timeline query not in exclusive lists (#4502)
this caches the stage of the home timeline query in which we calculate which account IDs should be shown in a particular user's timeline.
Reviewed-on: https://codeberg.org/superseriousbusiness/gotosocial/pulls/4502
Co-authored-by: kim <grufwub@gmail.com>
Co-committed-by: kim <grufwub@gmail.com>
Diffstat (limited to 'internal/cache')
| -rw-r--r-- | internal/cache/cache.go | 1 | ||||
| -rw-r--r-- | internal/cache/db.go | 15 | ||||
| -rw-r--r-- | internal/cache/invalidate.go | 7 |
3 files changed, 23 insertions, 0 deletions
diff --git a/internal/cache/cache.go b/internal/cache/cache.go index 2cc07de96..5bfa70ffa 100644 --- a/internal/cache/cache.go +++ b/internal/cache/cache.go @@ -96,6 +96,7 @@ func (c *Caches) Init() { c.initFollowRequest() c.initFollowRequestIDs() c.initFollowingTagIDs() + c.initHomeAccountIDs() c.initHomeTimelines() c.initInReplyToIDs() c.initInstance() diff --git a/internal/cache/db.go b/internal/cache/db.go index 8a8f59539..d5f25516b 100644 --- a/internal/cache/db.go +++ b/internal/cache/db.go @@ -145,6 +145,10 @@ type DBCaches struct { // FollowingTagIDs SliceCache[string] + // HomeAccountIDs provides access to the account IDs present in an account's + // home timeline, bearing in mind that some may be part of exclusive lists. + HomeAccountIDs SliceCache[string] + // Instance provides access to the gtsmodel Instance database cache. Instance StructCache[*gtsmodel.Instance] @@ -883,6 +887,17 @@ func (c *Caches) initFollowingTagIDs() { c.DB.FollowingTagIDs.Init(0, cap) } +func (c *Caches) initHomeAccountIDs() { + // Calculate maximum cache size. + cap := calculateSliceCacheMax( + config.GetCacheHomeAccountIDsMemRatio(), + ) + + log.Infof(nil, "cache size = %d", cap) + + c.DB.HomeAccountIDs.Init(0, cap) +} + func (c *Caches) initInReplyToIDs() { // Calculate maximum cache size. cap := calculateSliceCacheMax( diff --git a/internal/cache/invalidate.go b/internal/cache/invalidate.go index c6c25d4eb..58c427050 100644 --- a/internal/cache/invalidate.go +++ b/internal/cache/invalidate.go @@ -155,6 +155,9 @@ func (c *Caches) OnInvalidateFollow(follow *gtsmodel.Follow) { // results for them as mute / visibility result requester. if follow.Account == nil || follow.Account.IsLocal() { localAccountIDs = append(localAccountIDs, follow.AccountID) + + // Also invalidate their home account IDs cache. + c.DB.HomeAccountIDs.Invalidate(follow.AccountID) } // If target is local (or uncertain), also invalidate @@ -261,6 +264,10 @@ func (c *Caches) OnInvalidateList(list *gtsmodel.List) { // follow IDs in list. "f"+list.ID, ) + + // Invalidate user's home account IDs cache, + // as list exclusivity flag may have changed. + c.DB.HomeAccountIDs.Invalidate(list.AccountID) } func (c *Caches) OnInvalidateMedia(media *gtsmodel.MediaAttachment) { |
