element-hq / element-hq/synapse

synapse tracks lazy_loaded_members_cache per user/device but it should be per stream of events

Open
#5,421 0 comments 0 reactions 0 assignees View on GitHub
A-Lazy-Loading T-Defect
Dominant language
Python
Stars
4.6k
Forks
600
Avg merge
5d 22h
Merged PRs (30d)
51

Description

This issue has been migrated from [#5421](https://github.com/matrix-org/synapse/issues/5421).

---

i.e. if we stripped redundant events in /messages, it should use a separate cache to /sync. whereas the current code assumes they share a cache

Contributor guide

Open the contributing guide

Research direction

Search for lazy_loaded_members_cache and trace how it is used by the /messages and /sync event streams. Confirm whether redundant events are stripped through shared state, then separate the cache state per stream and verify that each stream tracks its own lazy-loaded members independently.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.