element-hq / element-hq/synapse
Add `limit` to `get_current_hosts_in_room`
Open
A-Federation
A-Messages-Endpoint
A-Performance
O-Uncommon
S-Tolerable
T-Enhancement
- Dominant language
- Python
- Stars
- 4.6k
- Forks
- 600
- Avg merge
- 5d 22h
- Merged PRs (30d)
- 51
Description
This issue has been migrated from [#13659](https://github.com/matrix-org/synapse/issues/13659).
---
*Follow-up to https://github.com/matrix-org/synapse/pull/13575. Part of https://github.com/matrix-org/synapse/issues/13356*
---
> To improve things further we could add a `limit` parameter to `get_current_hosts_in_room`. Realistically, we don't need 4k domains to choose from because there is no way we're going to query that many before we a) probably get an answer or b) we give up.
>
> *-- Going further section in https://github.com/matrix-org/synapse/pull/13575*
This means we can do less work in the database which means less waiting to get a list of domains to try.
Contributor guide
Assessment
This issue has not been assessed yet.