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

Lazy load room summaries from the cache #4384

Closed
manuroe opened this issue Jun 3, 2021 · 0 comments · Fixed by #5023
Closed

Lazy load room summaries from the cache #4384

manuroe opened this issue Jun 3, 2021 · 0 comments · Fixed by #5023

Comments

@manuroe
Copy link
Member

manuroe commented Jun 3, 2021

This task is more complex than #4382 and #4383 because we need a smarter storage for summaries. We want to be able to do smarter queries to built different kinds of room lists.

There will be a new and dedicated room summary storage. We will use a db to be able to fetch list of rooms in different manners.

We want to kill the 572ms in this log:

[MXFileStore] Start data loading from files
[MXFileStore] Loaded room messages of 625 rooms in 828ms
[MXFileStore] Loaded rooms summaries data of 625 rooms in 572ms
[MXFileStore] Loaded rooms account data of 625 rooms in 359ms
[MXFileStore] Loaded read receipts of 625 rooms in 1035ms
[MXFileStore] Loaded 2164 MXUsers in 71ms
[MXFileStore] Loaded 2 MXGroups in 1ms
[MXBaseProfiler] Task startup - storePreload for 625 units completed in 2870.760ms
[MXFileStore] Data loaded from files in 2871ms
@manuroe manuroe added this to the Sprint - Element 1.4.1 milestone Jun 3, 2021
stefanceriu added a commit to matrix-org/matrix-ios-sdk that referenced this issue Oct 25, 2021
…lter options structs. Removed fetch options references from them and made them equatable. Comparing them in the fetch options before refreshing the fetchers.
stefanceriu added a commit that referenced this issue Oct 25, 2021
stefanceriu added a commit to matrix-org/matrix-ios-sdk that referenced this issue Oct 25, 2021
…lter options structs. Removed fetch options references from them and made them equatable. Comparing them in the fetch options before refreshing the fetchers.
stefanceriu added a commit that referenced this issue Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant