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

[Version 3] Handling of sync/async relationships #81

Open
frederikbosch opened this issue Nov 16, 2018 · 2 comments
Open

[Version 3] Handling of sync/async relationships #81

frederikbosch opened this issue Nov 16, 2018 · 2 comments

Comments

@frederikbosch
Copy link
Contributor

frederikbosch commented Nov 16, 2018

Please take into account that this library uses a key/value store as backend. From our experience saving records in a separate key caused a gigantic load when it involved many records. Then for every relationships the adapter had to talk to localforage. This made the app unusable. Therefore we introduced a cache and caching setting at the level of the adapter. You can choose between fetching everything at once, fetching per model type or fetching per model. See the README.md.

Maybe this can be improved, or replaced by new/better methodology.

@NullVoxPopuli
Copy link

I like the cache approach.
Have you tested it with > 10k records or other large numbers of things being accessed?

@frederikbosch
Copy link
Contributor Author

Our app has around 1k-2k records. I believe I did testing back then with many more records. But memory was eventually becoming a problem. Especially because it had to perform on mobile phones from 6 years ago. So the answer of the performance is, of course, dependant on the use case. Another benchmark would be nice.

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

2 participants