-
Notifications
You must be signed in to change notification settings - Fork 0
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
Enable mysql and sqlite #34
Comments
SQLite complicates by treating dates as normal text. So there are 5 different types. Knex uses integer, (maybe it's temporary bug), sequelize uses custom format, CURRENT_TIMESTEMP returns without ms, it's a mess |
Maybe simplest solution is to check value when reading. This is done in sequelize, that I have to fix, since I think it does not support CURRENT_TIMESTAMP. Instead of new date, I can use knex.fn.now, maybe? |
If I want sqlite support, I certainly have to remove knex, since it's causing a lot of problems having 2 connections. And some weird behavior with transactions, it says it's busy. No problem in pg |
It also means converting migrations from |
SQLite would make it easier to start new project without docker.
There is good enough coverage that most of the things should work. I think only getting composite unique key is pg specific, and that can be easily ported.
The text was updated successfully, but these errors were encountered: