Skip to content

future of better-sqlite3 vs node:sqlite #1234

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

Closed
forgotPassword opened this issue Aug 7, 2024 · 2 comments
Closed

future of better-sqlite3 vs node:sqlite #1234

forgotPassword opened this issue Aug 7, 2024 · 2 comments

Comments

@forgotPassword
Copy link

Node 22.5 exposed sqlite module https://nodejs.org/api/sqlite.html, which also uses sync api. Will this affect the future of the lib?

Would be interesting to test it in the benchmarks.

@mceachen
Copy link
Member

mceachen commented Aug 8, 2024

I think it’s great to have more options! @JoshuaWise and I commented in the main PR that this feature dropped into, and they decided to adopt much of this library’s API signature.

That said, a specific version of SQLite will be baked into a given version of node, and there are substantive differences between versions that could result in frustration of you couldn’t pull in the SQLite version that has a newer bug fix.

@Prinzhorn
Copy link
Contributor

I had no idea this was added to Node.js! Right now this looks like a basic PoC that supports the bare minimum. And it looks like that's exactly the point. As a heavy user of user-defined functions and virtual tables I doubt the integrated SQLite will ever replace better-sqlite3. So my take is that for advanced use-cases better-sqlite3 is needed, but for basic CRUD tasks the integrated SQLite is perfectly fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants