-
Notifications
You must be signed in to change notification settings - Fork 987
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
Improve order of synced chats #12783
Comments
@cammellos not sure how the members list works exactly. Are updates send to a dedicated topic for online status updates? If so, we'd need to discuss where it sits in the fetching priority list (cc @simonricoo ) |
With ~40 pub chats (those which contain spam) after 1h offline it took 40s for 1-1 message be to received after opening chat's screen (that was the last opened chat, so it was opened right after login). |
1h, [42 4 15 18 7 12 5 7 9 15], avg=13.4 |
24h, [32 10 25 23 29 27 28 34 30 13], avg=25.1 |
After three days I just stopped receiving messages in that 1-1. Even if I send new ones, they are not received. Also not all outgoing messages were received on the other side, but idk whether that's because of mailservers. Basically 1-1 chat is broken. |
@rasom could you get |
@cammellos sure |
btw all 1-1 chats and private groups stopped working. I tried to add that account to a private group or to a new 1-1 chat and messages were not received. Pubchats work as expected. |
@cammellos https://gist.github.com/rasom/4f38aa50cd54dafe99a3b2d42cea1dc3 should I run it with |
@rasom not for now, I will ask you for more |
@rasom this can be closed for now? |
@cammellos agree |
As a user
I want to have chat synced in order
So that the most relevant messages are received in a shorted time interval
Description
Currently once a user goes back online, we fetch all the topics for all the chats in one go.
We should improve the way syncing works by being selective on which chat to sync first.
In order of importance we should sync:
Considerations
Some of these only make sense if the interval we are syncing is large enough (1 seems to be almost always a good choice, but after that we probably want to make sure it's worth the overhead, i.e if you are syncing 2 minutes of data, is most likely better to just sync everything in one go, as opposed if you are syncing a couple of hours).
The goal of this issue is to improve syncing, to bring the most relevant messages to the user as quickly as possible, so we should re-evaluate as we go along that it is indeed making things faster, and tests need to be performed with a device with many chats/contacts.
The text was updated successfully, but these errors were encountered: