-
Notifications
You must be signed in to change notification settings - Fork 238
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
[SQLite Dialect] Properly translate .ignore() #916
Comments
Hey 👋 Nice catch! @koskimas I wonder what the solution should be. The low hangin' fruit is just adding an override in WYSIWYG alternatives are: db.insertInto('person').or('abort')
db.insertInto('person').or('fail')
db.insertInto('person').or('ignore')
db.insertInto('person').or('replace')
db.insertInto('person').or('rollback') db.insertInto('person').orAbort()
db.insertInto('person').orFail()
db.insertInto('person').orIgnore()
db.insertInto('person').orReplace()
db.insertInto('person').orRollback() But having |
It's not wysiwyg but it's so close that I could live with it. But supporting all those other We might still override |
I definitely think that's the nice thing to do. |
Currently
.ignore()
using the SQLite dialect gets translated intoINSERT IGNORE INTO ....
Which doesn't match the SQLite expected
INSERT OR IGNORE INTO
(see here)The text was updated successfully, but these errors were encountered: