Replies: 1 comment 1 reply
-
seems like ALTER TABLE DROP COLUMN was added in 3.35.0, while ALTER TABLE ADD COLUMN was added in 3.2.0. We would accept a PR for the change. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hey, hope you are doing good. I was going to send a PR but I thought it's better to ask.
Is there a reason why the metadata returns false for
supportsAlterTableWithDropColumn
andsupportsAlterTableWithAddColumn
?I can execute these DDL without issues if I try, besides the known restrictions of sqlite. So it seems like these are supported operations.
Context:
I'm using Exposed for a project. I'm trying to implement migrations using Exposed Migration tools, and these tools use
supportsAlterTableWithDropColumn
to decide whether or not a migration should drop columns. So currently that is not working with this driver. Exposed PR.So in short, is a valid bug or is there a reason why these methods should return false?.
Thanks!
Beta Was this translation helpful? Give feedback.
All reactions