Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Member list in large rooms takes a very long time to load #851

Closed
kittykat opened this issue Jul 12, 2023 · 2 comments
Closed

Member list in large rooms takes a very long time to load #851

kittykat opened this issue Jul 12, 2023 · 2 comments
Assignees
@jmartinesp
Labels
A-Performance A-Room-Detail O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems X-Maybe-Release-Blocker Potential release blocker X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue

Comments

@kittykat
Copy link
Contributor

Steps to reproduce

  1. Go to Matrix HQ
  2. Click on the room header, then onto "People"

Outcome

What did you expect?

I can see member list

What happened instead?

I see a spinner for at least 15 minutes

Your phone model

Pixel 6a

Operating system version

Graphene OS

Application version and app store

0.1.0-nightly (1001000) - 12th July

Homeserver

matrix.org

Will you send logs?

Yes

Are you willing to provide a PR?

No
@kittykat kittykat added S-Critical Prevents work, causes data loss and/or has no workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems O-Occasional Affects or can be seen by some users regularly or most users rarely A-Room-Detail A-Performance Z-Schedule labels Jul 12, 2023
@jmartinesp
Copy link
Member

This is a Rust SDK issue: matrix-org/matrix-rust-sdk#1979

@manuroe manuroe mentioned this issue Jul 17, 2023
Closed
@kittykat kittykat added the X-Maybe-Release-Blocker Potential release blocker label Aug 24, 2023
@manuroe manuroe added the X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue label Aug 31, 2023
@Hywan Hywan self-assigned this Sep 6, 2023
@manuroe manuroe mentioned this issue Oct 19, 2023
Closed
3 tasks
@ganfra ganfra mentioned this issue Jan 3, 2024
Closed
@giomfo giomfo mentioned this issue Feb 23, 2024
Closed
@manuroe manuroe assigned jmartinesp and unassigned Hywan Mar 12, 2024
@jmartinesp
Copy link
Member

I just tested this in a debug build with this PR in my old test device and the loading times including the network request and response processing time went from 12 minutes in the Matrix HQ room (with ~110.000 users) to 45s. In release mode this should be even faster.

However, we still have an issue with the /members request failing and somehow bot being retried again, although according to the Rust code it should. This happened while testing and the only solution for me was to restart the app and then try to reload the member list for the room. I'll open an issue in the Rust SDK repo.

There is also some work needed to be done now to improve the UX of the room member list screen, but that's out of the scope of this issue I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmartinesp jmartinesp

Labels
A-Performance A-Room-Detail O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems X-Maybe-Release-Blocker Potential release blocker X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kittykat @jmartinesp @Hywan @manuroe