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

Synchronous Callbacks #642

Closed
ccorcos opened this issue Feb 28, 2019 · 3 comments
Closed

Synchronous Callbacks #642

ccorcos opened this issue Feb 28, 2019 · 3 comments

Comments

@ccorcos
Copy link

ccorcos commented Feb 28, 2019

Why aren't callback called synchronously? Its a shame that level-mem can't work work synchronously as an in-memory cache.

https://github.com/Level/mem

@vweevers
Copy link
Member

Why aren't callback called synchronously?

For consistency with other stores, and to dezalgo.

Its a shame that level-mem can't work work synchronously as an in-memory cache.

Synchronous variants of methods (e.g. getSync) have also been suggested for leveldown: Level/leveldown#372. We're open to it.

@ccorcos
Copy link
Author

ccorcos commented Feb 28, 2019

Neat-O. I will try to see what I can do.

@ccorcos
Copy link
Author

ccorcos commented Feb 28, 2019

Looking through everything, there are so many packages that its going to be a pain to make everything work synchronously. In the meantime, I tried a more "engineered" solution for making level-mem synchronous and it works quite well... 😄

Level/mem#50

@ccorcos ccorcos closed this as completed Feb 28, 2019
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