Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

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

Open
ara4n opened this issue Jun 10, 2019 · 0 comments
Open

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

ara4n opened this issue Jun 10, 2019 · 0 comments
Labels
A-Lazy-Loading Issues relating to `lazy_load_members` in `/sync` T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@ara4n
Copy link
Member

ara4n commented Jun 10, 2019

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
@turt2live turt2live mentioned this issue Jun 10, 2019
Merged
@neilisfragile neilisfragile added z-p2 (Deprecated Label) z-bug (Deprecated Label) labels Jun 13, 2019
@richvdh richvdh added T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. A-Lazy-Loading Issues relating to `lazy_load_members` in `/sync` and removed z-bug (Deprecated Label) z-p2 (Deprecated Label) labels Jun 8, 2022
@matrixbot matrixbot mentioned this issue Dec 21, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Lazy-Loading Issues relating to `lazy_load_members` in `/sync` T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@neilisfragile @ara4n @richvdh