-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add link to "official" fork #682
Comments
Another good link to be added is https://github.com/bajankristof/nedb-promises#readme , since callbacks are you know, a little uncomfortable. |
This might also be worth considering: https://github.com/nedbhq/nedb-core Courtesy of https://andremiras.github.io/gitpop3/ |
Also perhaps https://github.com/typicode/lowdb deserves a mention as a solid alternative |
@RollingHog We released a version of our fork which removes the use of https://github.com/seald/nedb/blob/master/CHANGELOG.md#300---2022-03-16 |
@tex0l why did you choose the promise-based executor? |
Because doing it without |
With Chez and Gambit callback-based code is ten times faster, than colored |
At Seald (for which we maintain this fork), we don't observe any performance issue with our implementation. If you want to do a benchmark between the historical version of nedb & our latest version with async code, feel free. I don't think there is a 10x difference. Even if there is on some environments, as the whole JS ecosystem moves towards async code, I don't think going back to callbacks is a good idea. The subsequent versions of node.js & browsers will catch up the difference if there's any. |
Since this won't be supported anymore, maybe owner should add a link to what would be recognized as a "official" and stable fork of this repo, like https://github.com/seald/nedb/ ?
The text was updated successfully, but these errors were encountered: