Fixing missing room summary last messages slows down the initial sync #6240
Assignees
Labels
A-E2EE
O-Occasional
Affects or can be seen by some users regularly or most users rarely
S-Major
Severely degrades major functionality or product features, with no satisfactory workaround
T-Defect
Something isn't working: bugs, crashes, hangs and other reported problems
Comments
manuroe
added
the
T-Defect
stefanceriu
added
A-E2EE
S-Major
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps to reproduce
After this PR, last year, an initial sync required 467 decryptions on a 600 room account, using 2.8s of an iPhoneX CPU.
1.8.17 requires now 4643 decryptions for a 850 room account. They use 120s of iPhone 13 CPU before this fix for #6239 (Maybe 25s now).
But we are still comparing 2.8s of iPhoneX with 25s of iPhone 13.
The change comes from matrix-org/matrix-ios-sdk@1f9d638?diff=split&w=1 that transforms the lazy fix of room summaries by a mandatory step.
This step is expansive. It fetches (and decrypts) 50 messages per problematic rooms.
Outcome
Reduce the number of required decryptions during an initial sync.
Your phone model
No response
Operating system version
No response
Application version
No response
Homeserver
No response
Will you send logs?
No
The text was updated successfully, but these errors were encountered: