-
-
Notifications
You must be signed in to change notification settings - Fork 654
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
Long, sometimes endless loading in message list #4033
Comments
Hi @maltokyo, thanks for the report! Here's a screenshot from the video, to make it easier for people looking at this issue to see what you're seeing: I don't have a clear diagnosis for this yet, but we know of one issue that could cause long wait times in hard-to-reproduce ways. @gnprice or @ray-kraesig, I think #3841 is getting close to mergeable and is awaiting review. I'm not saying this is definitely the solution to this problem, but it's kind of hard to rule out except empirically. |
Great. Thank you. The other thing I can say is that recently when it happens on mobile app I've always had web version nearby. And it was always super responsive without issue while the mobile app was was behaving like this. Both on the same WiFi network. So again, I did everything I could to isolate it to be sure it's an issue with the app. Any log I can attach? |
#3841 was merged today, so it should be out in the next beta, probably next week; you can watch for that to be released here in the Zulip developer community, but I'll post again here when that happens, if I remember. And, in any case, we'll keep this issue open until we're sure it's been fixed. Or, if you'd like to test it sooner, you can build and run the app from the latest code on the |
Thanks a lot @chrisbobbe ! So nice to hear responses like this and motivates people to submit issues more when they find. I'll try the beta next week or when available. |
So, there was a beta release yesterday, the 13th, but it only had one cherry-picked fix, deemed critical because the problem was a total crash of the app on startup, and that release it didn't include the attempted fix for this issue. @gnprice, we'll plan to do a regular beta release with everything on master this week, I think, right? |
Hi @chrisbobbe & all
https://www.youtube.com/watch?v=1RqD1X-9j1Y&feature=youtu.be
https://www.youtube.com/watch?v=rZMjHYoRFhE&feature=youtu.be
https://www.youtube.com/watch?v=Qvpjr9kSCmY&feature=youtu.be These videos are "unlisted", and I will delete soon, please acknowledge that you have seen the problem(s) enough for me to bring them down. |
Thanks for the additional reports, @maltokyo!
|
Hi @chrisbobbe Regarding #1 I've frequently left it for some minutes like that (not 5 mins for sure, but at least 1:30 - 2:00). For me it doesn't resolve itself and I just restart the app which fixes it 100% of the time. Next time it happens I'll leave it for 10min, see how it goes, and report back. Does the app make any logs at all, that would be useful for you guys? I'm happy to send if you let me know how to access. |
There's been another report of something that looks quite similar, also on Android, here. That report said it was happening to multiple other people, on multiple realms. @maltokyo, thanks for the offer to include logs! The app does indeed produce logs, but we should be able to see any relevant ones ourselves, in Sentry. We'll let you know if not, though. @gnprice, @ray-kraesig, do you know a way to filter our Sentry logs to see errors whose first occurrence is recent, and that only happen on Android? |
Hmm, @gnprice, @ray-kraesig, take a look at this filtering/sorting in Sentry. That's a summary of events that have occurred at least once in the last 14 days, where the first-seen date for that class of events (I know we have some open issues about (dis)aggregation, like #3864) is within the last three weeks, sorted by frequency. The first four are for the error message However, there's an interesting result that's popping up, and is showing what may be an upward trend in occurrences; it certainly seems that way more than other event classes in these search results:
The error message (which, again, we haven't seen in Sentry before) is "RuntimeException: Probable deadlock detected due to WebView API being called on incorrect thread while the UI thread is blocked." Looks like it was first seen on 26.25.148 (GitHub, CZO). Just opened this as #4051. |
@maltokyo , are you continuing to see this symptom? I have a hypothesis that it might be tied to the Chrome version used for the WebView, in part because the timing (especially in the graph Chris posted just above) lines up with when a new Chrome release came out. If you are still seeing the issue, you might be able to help us pin that down. Two quick questions first:
Then, if you're up for an experiment: I would be very interested in the results if you try opting into the "dev" channel of Chrome (currently Chrome 84) for your WebViews. Instructions here: |
Hi @gnprice |
This issue also affects (all of?) our staff in the company. Both on Android as well as iOS. We do have some streams that load, but most are just (at least seemingly) loading forever. The ones that do load only have very few messages in them. The desktop and web clients are working perfectly. Previous messages in this ticket are now weeks old already. Has there been any movement or the possibility to get/test a beta release? |
@pohutukawa Thanks for the report! Please file it as its own issue so we can debug it properly -- it sounds like it's a different issue from this one, and I have some followup questions I'll be interested in asking to learn more about what you're seeing. |
This issue is now back for me too. I didn't experience it for some time
until the last few days. Same as before.
…On Sat, 13 Jun 2020, 01:08 Greg Price, ***@***.***> wrote:
@pohutukawa <https://github.com/pohutukawa> Thanks for the report! Please
file it as its own issue so we can debug it properly -- it sounds like it's
a different issue from this one, and I have some followup questions I'll be
interested in asking to learn more about what you're seeing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4033 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKS7W6R2TVMTMXB7WMHCQDRWKYNRANCNFSM4MFOJ6JA>
.
|
@maltokyo Very interesting. Thanks for the update, and sorry the issue seems to be back. I asked some questions at #4033 (comment) that may help us debug the issue. Now that you're seeing the issue again, I'd be very curious for the answers. |
Hi @gnprice Please find below, sorry for the delay.
MIUI 11, based on Android 10 (Xiaomi, latest updates)
Version 83.0.4103.106 (updated on 15th June)
I tried this, however, due to the fact that I have a "work profile" installed, it will not let me select the second line in this screenshot, any way around it? |
Cool -- so that's consistent with our guess that this symptom may correspond to the error #4051 we're seeing reported as an exception, because that error is only possible on Android 10+. (Though something like half our Android users are on Android 10 -- 38% when I last looked a couple of months ago -- so one data point isn't super strong confirmation by itself.)
That's helpful, thanks.
Hmm. I don't know very much about how a "work profile" works, but I would guess that this means that this is a device managed by your employer's IT department, and it's saying that changing the Chrome version to be used for webviews is something off-limits for you to do yourself -- that it can only be done by someone with those IT privileges. (Which is pretty reasonable; keeping everyone's configurations uniform helps keep down the complexity of debugging the IT staff may have to do.) If you have a good relationship with someone in that department who might be up for helping you debug this issue you're seeing in Zulip, you might ask them to make this change for you. I would certainly be grateful to see the results. If it helps, the version I'm most interested in -- Chrome 84 -- is now in the Beta channel, so you can use that instead of Dev. Slim chance that'll give any different result on this screen; but if you do ask someone in your IT department for help, they might feel more OK about a beta version than a dev version. |
@maltokyo One other question for you (or pair of questions), which I should have asked sooner but I see I neglected to!
|
Hi @gnprice I only have tried with my own (debian 10, 32GB RAM, VPS) server using docker version of Zulip, but I can give it a go with the main zulip chat site as well. Below is the outcome of running that command on my server (domain replaced).
|
That's helpful information, thanks.
(Unrelated to this issue, I'd recommend you upgrade the server -- there was a security release last month.) |
@maltokyo Have you been continuing to see this symptom? We've been seeing in Sentry a much lower rate of the exception #4051 which looked like it might have been related, ever since around the time of the last few comments on this thread. So I'm hoping that you've also stopped seeing this, or at least that it's become rare. |
We'd marked this as P1 last year because it's a bad symptom, and it looked like it might be related to #4051 which was an exception report we were getting from a lot of users so it seemed likely that a lot of users were seeing this. Since then #4051 has become infrequent, and we haven't had any reports of this from additional users, so that no longer seems likely. |
Hi Greg
It's become rare indeed but still happens. I'd say once in 10-20 times I
open the app. Very strange.
…On Wed, 14 Apr 2021, 02:44 Greg Price, ***@***.***> wrote:
We'd marked this as P1 last year because it's a bad symptom, and it looked
like it might be related to #4051
<#4051> which was an
exception report we were getting from a lot of users so it seemed likely
that a lot of users were seeing this.
Since then #4051 <#4051> has
become infrequent, and we haven't had any reports of this from additional
users, so that no longer seems likely.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4033 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKS7W6Z7LUJFHGNPHVQZQ3TITQO7ANCNFSM4MFOJ6JA>
.
|
When I login, from time to time (I would say one in five times I open the app) I get this "loading" screen. Sometimes it lasts for 5 seconds, sometimes for 30 seconds.
Restarting the app (force quit and reload) fixes it immediately 100% of the time, so I am sure there is nothing wrong on server side.
Video is attached, I could not upload the mp4 file directly, so it is zipped here.
Screenrecorder-2020-03-26-22-04-19-301.mp4.zip
The text was updated successfully, but these errors were encountered: