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

Search functionality on Apps Managements screen is crazy slow/laggy #2951

Closed
carlylemiii opened this issue Jul 24, 2019 · 8 comments
Closed
Assignees
Milestone

Comments

@carlylemiii
Copy link

carlylemiii commented Jul 24, 2019

Steps to reproduce

  1. Open Apps Management
  2. Tap Search app… button
  3. Notice Search is slow to load.
  4. Tap the name of an app you have installed.
  5. Watch as what you've typed at a normal speed finally catches up and is displayed in AdGuard.
  6. Tap on the result.
  7. Result is slow to load, and before loading, you're shown the full list of apps again in Apps Management.
  8. Search for another app.
  9. Tap back arrow and notice that backwards navigation is even slow.
  10. Tapping an app from the full list is even a little slow - not painfully so, not like in the above scenarios, but enough that it's a noticeable delay.

Expected behavior

Speed!

Actual behavior

Sooo slow it's like I'm running the app on a device from five years ago.

Your environment

  • Adguard version: 3.2.98β
  • Adguard filtering mode: Local VPN
  • Device model: Pixel 3 XL
  • Operating system and version: Android 9
  • Root access: No
  • Any other apps that can start VPN / Antiviruses / Cleaners etc. installed?: No
@zzebrum
Copy link
Contributor

zzebrum commented Jul 24, 2019

I agree with step 5. Other steps look not that bad on my device though.
@TheHasagi please check on other devices.

@carlylemiii
Copy link
Author

Let me know if you need video of any of the others. 2-3 should be painfully obvious, too, the rest are less drastic but still quite reproduceable.

@ameshkov
Copy link
Member

Actually, it is really very slow.

@artemiv4nov please check what's done when the search is performed.

I am starting to think that we are re-executing SQL queries for the apps' stats every time the text is changed in the search input. This may be fast for newly installed apps, but when you have a lot of stats to select, this will be super slow.

@artemiv4nov artemiv4nov assigned Revertron and unassigned artemiv4nov Jul 25, 2019
@carlylemiii
Copy link
Author

I saw this mentioned as an improvement in the release notes for v3.2.117β but I cannot tell a difference at all for what it's worth.

@ameshkov
Copy link
Member

ameshkov commented Aug 4, 2019

@carlylemiii could you please try the nightly build? I am on .119 and I don't see any issues.

@carlylemiii
Copy link
Author

@ameshkov it is still very slow in the nightly. Note how long it takes for the search to open after I press the search button (that's not me long-pressing; that's just it taking a while to respond). Then, note how much of a delay there is between my keyboard input and that being displayed in the search bar (including backspacing, as well). Point number 7 above still is true, too, though it's not captured in this recording I just made, as I skipped that part.

https://www.dropbox.com/s/w3ghdsh9cz7mop9/20190805_002757.mp4?dl=0

@carlylemiii
Copy link
Author

Probably had to do with the large number of stats I had accumulated. I cleared storage for the app, and App Management is pretty speedy now. So, issue resolved for me, for now, but the issue likely still actually exists.

@carlylemiii
Copy link
Author

@ameshkov I'd love to see this reopened. Things are still very slow, almost not any different than when I originally reported.

@ameshkov ameshkov modified the milestones: 3.2, 3.3 Oct 23, 2019
@ameshkov ameshkov reopened this Oct 23, 2019
@ameshkov ameshkov assigned artemiv4nov and unassigned Revertron Oct 23, 2019
@artemiv4nov artemiv4nov assigned Revertron and unassigned artemiv4nov Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants