-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 db migration fails on table already existing #224
Comments
@rgarcia thanks! @fudanchii will merge on your approval |
Yeah, we should do this. if err := rev.Up(mg); err != nil {
log.Printf("Failed to upgrade to Revision Number %v\n", current)
log.Println(err)
// suppress any possible error from tx.Rollback
tx.Rollback()
return err
} But then I revert this and going with the old behavior, since this is will useful only if we extract the migration system from the main app (standalone application to do migration solely). This could be merged in, I will fix mysql to adapt this. Thanks. |
Trying to upgrade to the latest version of drone and seeing this error on startup:
I changed
to use
CREATE TABLE IF NOT EXISTS
, which seems more accurate. I'll submit a PR shortly.The text was updated successfully, but these errors were encountered: