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

v10.1.1: fix import/export operations on iOS #180

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

JaffaKetchup
Copy link
Owner

@JaffaKetchup JaffaKetchup commented Feb 3, 2025

  • Converted import/export operations to use chunking instead of streaming
  • Improved efficiency of import/export operations

Fixes #178.

@JaffaKetchup
Copy link
Owner Author

The problem is that using manual chunking seems to grow the database rapidly for some reason, and uses considerably more memory, whereas streaming seems to be slightly slower, but uses much less memory and does not appear to grow the database at all.

Unfortunately, the growth caused by the chunking is quite significant (it seems to be the entire amount of the import stores' tiles), and that might make this unsuitable for production, especially as it cannot be shrunk afterward.

Remove unneccessary check on each iteration in `chunked`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Import/export operations causing fatal crash on iOS
1 participant