1.4.2 file system polling battery drain

[maltklaus]

Actual behaviour

updating to 1.4.2 (rc4 or release) creates battery drain when photo upload is selected and watching an sd card. the processes com.nextcloud.client, sdcard and mount.exfat eat up cpu resources

Expected behaviour

no significant resource usage as in previous versions

Steps to reproduce

1. install 1.4.2
2. enable picture upload, wifi only, loads of pictures on sd card
3. use a process monitoring tool like better battery stats to loom at the processes

Environment data

Android version: 6.0.1

Device model: hlte

Stock or customized system: resurrection remix

Nextcloud app version: 1.4.2

Nextcloud server version:latest

Logs

Web server error log

n. a.

Nextcloud log (data/nextcloud.log)

would have to recreate

[tobiasKaminsky]

cc @mario as he rewrote the auto upload part

[mario]

Seems to be a particular issue with SD cards on certain phones, yes. Dunno how to debug, but will try.

[returntrip]

Same issue for me, battery drain caused by high CPU usage while the client is configured to auto upload. If I disable the auto upload, Nextcloud process stops draining battery and high CPU usage stops.

No matter if I enable uploads only while charging, Nextcloud would still show High CPU usage and battery drain.

Can provide screenshots or other info if needed.

Environment data

Android version: 7.1.2

Device model: Nexus 6P

Stock or customized system​: Fully stock, no root, locked bootloader

Nextcloud app version: 1.4.2

Nextcloud server version:latest

[mario]

This will be fixed in 1.5 I hope :) We will pool every half an hour or so.

…

On Sun, 30 Apr 2017 at 11:01, Stefano ***@***.***> wrote: Same issue for me, battery drain caused by high CPU usage while the client is configured to auto upload. If I disable the auto upload, Nextcloud process stops draining batter and high CPU usage stops. No matter if I enable uploads only while charging, Nextcloud would still show High CPU usage and battery drain. Can provide screenshots or other info if needed. Environment data Android version: 7.1.2 Device model: Nexus 6P Stock or customized system​: Fully stock, no root, locked bootloader Nextcloud app version: 1.4.2 Nextcloud server version:latest — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#746 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAWst28Vol8xndbUvTGcf37FXLChqvoks5r1E3rgaJpZM4Mdygr> .

[returntrip]

@mario thanks for the quick reply. Should I try the RC? Perhaps it is fixed there? Can the auto upload feature stop polling all together when the phone is not charging?

[mario]

It is not yet :) I have to rewrite auto upload once again :)

…

On Sun, 30 Apr 2017 at 11:08, Stefano ***@***.***> wrote: @mario <https://github.com/mario> thanks for the quick reply. Should I try the RC? Perhaps it is fixed there? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#746 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAWsj1YL-tUCzum-tBTNvns7aZ8Bh1hks5r1E9dgaJpZM4Mdygr> .

[returntrip]

@ OK! Would be great if you could disable checking for files to upload when the phone is disconnected from the charger and the "upload only while charging" function is ticked (instead of checking every 30 minutes). Let us know when you wish to test :).

[lexelby]

This may be a naive question, but why poll at all? Can't it just react to new photos and videos using the NEW_IMAGE and NEW_VIDEO intent receivers?

[mario]

Unfortunately no. It never worked on all phones, and starting Android 7 it doesnt work anywhere.

…

On Sun, 30 Apr 2017 at 14:07, Lex Neva ***@***.***> wrote: This may be a naive question, but why poll at all? Can't it just react to new photos and videos using the NEW_IMAGE and NEW_VIDEO intent receivers? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#746 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAWsnn3VOsEskDgrAYD162mcsnDb3F9ks5r1HmKgaJpZM4Mdygr> .

[lexelby]

Ugh, how annoying! I wonder why they would get rid of such a useful feature.

The problem is, I (like other folks here?) have like 5000 photos on my sd card. Polling would take a lot of processing power and i/o, I would expect.

[lexelby]

https://developer.android.com/about/versions/nougat/android-7.0-changes.html#bg-opt mentions this change, for those curious.

[mario]

@lexelby will be better when we start doing it every 30 minutes instead of every second :)

[snicke]

Can confirm this issue. On may Galaxy S7 Edge with Android 7.0 the system is always warning about one app draining the battery and ask if I want to put it in "sleep mode". The app: Nextcloud 1.4.2.

On a related note. If autoupload is turned on, it misses many files so the autoupload is unfortunately not reliable and useful today: #804

[GitHubGeek]

Confirmed this issue occurs on OnePlus One internal storage (virtual SD card). App's CPU usage is a constant 4-8%.

Suggest to watch for filesystem changes using an event-based mechanism, then scan/poll every 1 hour to catch any missing files. (All event based systems occasionally miss files)