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

indexeddb: move attachments out of DOC_STORE? #8811

Open
alxndrsn opened this issue Oct 27, 2023 · 0 comments
Open

indexeddb: move attachments out of DOC_STORE? #8811

alxndrsn opened this issue Oct 27, 2023 · 0 comments

Comments

@alxndrsn
Copy link
Member

Pouch docs:

What’s the point of attachments? If you’re dealing with large binary data (such as PNGs), you may incur a performance or storage penalty if you naïvely include them as base64- or hex-encoded strings inside your documents. But if you insert the binary data as an attachment, then PouchDB will attempt to store it in the most efficient way possible.

pouchdb-adapter-indexeddb's current implementation currently follows the naïve path.

Moving attachments e.g. to a separate IDBObjectStore would have performance implications, and would require a migration.

If making this change, care should be taken to re-use IndexedDB transactions where possible.

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

No branches or pull requests

1 participant