mattttvaughn / chronicle

GNU General Public License v3.0

216 stars 58 forks source link

Paginated sync to server #42

Closed mattttvaughn closed 2 years ago

mattttvaughn commented 2 years ago

Fetch books/tracks in batches of 100 each, as opposed to fetching all in a single request.
This will prevent timeout errors for enormous libraries and for those on slow internet connections

10 #2

mattttvaughn commented 2 years ago

If you have a large library (esp. a lot of individual tracks, say 100k+), would appreciate if you could pull this down and try it out.

I'm seeing reasonable memory usage for a library with 15k tracks, but as noted in the TODO in TrackRepository it'll hit an out of memory exception at some point. Hoping it'd have to be an unreasonably large library, otherwise will have to rework it some

pmayeur commented 2 years ago

Phone: Pixel 6 Pro (12GB Ram) Plex: Version 1.25.7.5604 Library: 67,266 audio files (77,013 with covers and txt files)

Testing: Turn on "Refresh Always". Listen to books. Manually pull down to refresh, while listening. Listen to books change refresh interval. Pause and Play. Close and open app.

LGTM

mattttvaughn commented 2 years ago

Link to a compiled apk based on this PR: https://github.com/mattttvaughn/chronicle/tree/feature/Paginated-Sync/app/release