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.

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

Bug: syncAllGroups takes longer than expected on first run on Android #511

Closed
alexrisch opened this issue Oct 17, 2024 · 1 comment · Fixed by #518
Closed

Bug: syncAllGroups takes longer than expected on first run on Android #511

alexrisch opened this issue Oct 17, 2024 · 1 comment · Fixed by #518
Assignees
Labels
bug Something isn't working

Comments

@alexrisch
Copy link
Contributor

Describe the bug

Slack Thread: https://xmtp-labs.slack.com/archives/C05GBHEAGPP/p1729097715935989

Seeing the first run of syncAllGroups taking a long amount of time: ~20-40 seconds

This seems to happen mostly for Android
Slack thread has logs from both app and libxmtp

Note this typically happens on the very first load, others will happen a lot quicker

Expected behavior

No response

Steps to reproduce the bug

No response

@alexrisch alexrisch added the bug Something isn't working label Oct 17, 2024
@nplasterer nplasterer self-assigned this Oct 17, 2024
@nplasterer
Copy link
Contributor

I've written up a test for this and It seems to be running quickly for me every time. You mind sharing some more details on the environment that is causing this? Possible other streams running? etc... I did see a really large slow down when we got up into 1000 groups that each had 10 new messages but even that was only 13 seconds on Android. We have some ideas for perf improvements for once this list gets larger but that seems unrelated to the issue you are having currently?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants