Upload Whole Camera Roll: OOM while indexing
Nobody has claimed this yet.
- Dominant language
- Swift
- Stars
- 2.5k
- Forks
- 1k
- Avg merge
- 2d 18h
- Merged PRs (30d)
- 13
Description
Steps to reproduce
- Enable
Upload Whole Camera Rollwith ~30k photos - Watch loading spinner
Expected behaviour
Eventually photos are indexed and added to the upload queue
Actual behaviour
On iPhone 13:
App crashes after a few minutes
On iPad Pro 5th gen:
Works as expected
The iPad has 16 Gb of RAM, opposed to the iPhone with 4 Gb. I suppose we run into a memory limitation.
Environment data
iOS version: iOS 15.1 on both devices
Nextcloud iOS app version: 4.0.6.0
Server operating system: NixOS 21.05
Web server: Nginx
Database: PostgreSQL
PHP version: 7.4.23
Nextcloud version: 22.1.1
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files, tests, or entry points are named. Start by reproducing whole-camera-roll indexing with about 30k photos on an iPhone 13, then trace the indexing path and compare memory behavior with the iPad Pro; done means the photos are indexed and added to the upload queue without the app crashing.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ios, swift
- Domain
- mobile
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100