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

Move /groups from master process #6415

Closed
neilisfragile opened this issue Nov 26, 2019 · 3 comments
Closed

Move /groups from master process #6415

neilisfragile opened this issue Nov 26, 2019 · 3 comments
Assignees
@erikjohnston
Labels
A-Performance Performance, both client-facing and admin-facing z-p2 (Deprecated Label)

Comments

@neilisfragile
Copy link
Contributor

neilisfragile commented Nov 26, 2019 •
edited by richvdh
Loading

No reason for it to be on master, should help CPU-bound homeservers running in worker mode.
@neilisfragile neilisfragile added A-Performance Performance, both client-facing and admin-facing z-p2 (Deprecated Label) labels Nov 26, 2019
@richvdh
Copy link
Member

richvdh commented Nov 26, 2019

this applies to groups apis other than /groups itself; /joined_groups may be a larger culprit

@neilisfragile
Copy link
Contributor Author

/joined_groups, /publicised_groups, /groups/.../profile, /groups/.../rooms

This was referenced Feb 6, 2020
Closed
Merged
@richvdh
Copy link
Member

richvdh commented Feb 20, 2020

fixed by #6866 I guess

@richvdh richvdh closed this as completed Feb 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@erikjohnston erikjohnston

Labels
A-Performance Performance, both client-facing and admin-facing z-p2 (Deprecated Label)
Projects
None yet
Milestone
No milestone
Development

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