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

Performance issue/hangs in 0.6.6 #217

Closed
mang1ed opened this issue Sep 21, 2020 · 17 comments
Closed

Performance issue/hangs in 0.6.6 #217

mang1ed opened this issue Sep 21, 2020 · 17 comments
Assignees
Labels

Comments

@mang1ed
Copy link

mang1ed commented Sep 21, 2020

0.6.6 fdroid version appears to freeze and or would take long during sync and or when loading keyboard/minimising the app (having to force close the app). I tried to reinstall the app and clearing data/cache. App works fine for the first few messages and then starts to hang. I had to revert to 0.6.5 as this one works great on my device. I'm running /e/ on a note 3.

@michaelkourlas
Copy link
Owner

michaelkourlas commented Sep 21, 2020

I've gotten multiple reports of this and the common factor seems to be older versions of Android. (The app works fine on my own device -- Pixel 2 XL running Android 11.)

Some changes made that might be related:

  • updated various support libraries
  • updated to build with Android 11 SDK
  • started using WorkManager for the first time, since IntentService was deprecated
  • started using okhttp3 instead of HttpURLConnection

Other than that, I don't know what the cause might be.

@mang1ed
Copy link
Author

mang1ed commented Sep 21, 2020

My device is running android 7.1.2. There could be an incompatability with the libraries like you've mentioned.

@baldurmen
Copy link

baldurmen commented Sep 21, 2020

I have similar issues running the 0.6.6 version of the app on Android 9. Certainly not the latest version, but I wouldn't say it's "old" :)

@michaelkourlas michaelkourlas self-assigned this Sep 27, 2020
@michaelkourlas michaelkourlas added this to the 0.6.7 milestone Sep 27, 2020
@LinearDynamics
Copy link

Also experiencing this issue from the recent update. In Android settings>Battery>Advanced, Voip.ms SMS app used 21% since last full charge. UI doesn't load message threads.
Android 10, OnePlus 6t(Oxygen 10.3.5).
In the meantime, I'm using the not so great web page for my SMS https://voip.ms/m/communications.php

@michaelkourlas
Copy link
Owner

Unfortunately, I'm at a loss. I'm just not experiencing these issues, so I don't know what's causing them.

@michaelkourlas michaelkourlas removed this from the 0.6.7 milestone Sep 28, 2020
@michaelkourlas
Copy link
Owner

A user said that rebooting fixed the problem with the latest version. I wonder...

At first I thought this was happening because there were multiple OkHttp client instances. That probably didn't help, but it sounds like fixing that didn't fix the underlying problem, which is some kind of memory leak.

Based on this StackOverflow question, I think this is because I'm not closing the HTTP connection.

Will quickly release an update to fix this.

@michaelkourlas
Copy link
Owner

Actually, never mind, I was already closing the response.

@michaelkourlas
Copy link
Owner

@LinearDynamics -- what happens if you restart your device? Does the problem go away for you too?

@LinearDynamics
Copy link

@LinearDynamics -- what happens if you restart your device? Does the problem go away for you too?

Unfortunately, no. I did try that yesterday. Also today. Performed an uninstall/reinstall(with login to the API) but phone became noticeably hot after each test. In a bit I was going to try to delete the database, uninstall, set the synch start to August and retest. Not sure if that would be a worthwhile effort.
When initially starting the app, I can(sometimes) load the sms threads and reply however after a minute the unresponsiveness begins to occur.

@michaelkourlas
Copy link
Owner

In a bit I was going to try to delete the database, uninstall, set the synch start to August and retest. Not sure if that would be a worthwhile effort.

It might be. This might be a scaling issue.

@LinearDynamics
Copy link

Ok, I set synch date to September 15,deleted dB, cleaned up.
In the past 10 minutes I'm able to send messages to my own DID. Works quickly and reliably.
My phone is not getting the heat blast like it was before.
So for now, I'll keep the synch start as is and report back.

@yulman19
Copy link

I'm not sure i'm in the good thread, but since last update i'm having a lot of troubles with the app similar to the ones mentionned with a fresh DB.
Android 11 on a Pixel 3a.

@michaelkourlas
Copy link
Owner

I'm just going to start using IntentService again. I don't care that it's deprecated -- the implementation is super straightforward. I'll just make my own copy in case they ever remove it.

@AMDSeb
Copy link

AMDSeb commented Sep 29, 2020

I'm just going to start using IntentService again. I don't care that it's deprecated -- the implementation is super straightforward. I'll just make my own copy in case they ever remove it.

Thanks a lot once again for your work. I was also experiencing very long waits when sending messages (1-2 minutes) since the last version update. Running Android 8 (annoying manufacturers, should be mandatory to keep updating for a certain period). I was about to post to see if I could help pinpoint the problem, but saw you had published a new version. Just tried it and it seems back to normal, no more hangs. Thanks again!

@michaelkourlas
Copy link
Owner

That's great to hear! If 0.6.8 fixed things for anyone else -- or if the problem's still there -- please let me know.

@LinearDynamics
Copy link

I extended my synch time back to 2018. All messages are present and performance seems normal.

@michaelkourlas
Copy link
Owner

I think the problem here, for reference, was that I was using WorkManager incorrectly -- the work was running on the UI thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants