element-hq / element-hq/synapse
synapse tracks lazy_loaded_members_cache per user/device but it should be per stream of events
Open
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
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