Skip to content
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

Repair: also correct migration type #1804

Closed
wanglin0413 opened this issue Oct 24, 2017 · 2 comments
Closed

Repair: also correct migration type #1804

wanglin0413 opened this issue Oct 24, 2017 · 2 comments

Comments

@wanglin0413
Copy link

@wanglin0413 wanglin0413 commented Oct 24, 2017

What version of Flyway are you using?

flyway-4.2.0

Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)

Command-line

What database are you using (type & version)?

Mysql 5.8

What operating system are you using?

windows 10

What did you do?

(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
I wrote a V38__Change.sql and executed migrate,but now I want to change V38 to jdbc migrate, because it's difficult to implement new requirement with sql. With repair I can only change checksum, but cant change type.

What did you expect to see?

With a command I can change version type instead of modifying with mysql states handly. and databases which already migrated to version higher than V38, should ignore new Jdbc-type V38 other than fail.

What did you see instead?

There is no such command.

@axelfontaine axelfontaine changed the title how to change schema_version type from sql to jdbc with repair? Repair: also correct schema_version type Oct 24, 2017
@axelfontaine axelfontaine added this to the Flyway 5.0.0 milestone Oct 24, 2017
@axelfontaine axelfontaine changed the title Repair: also correct schema_version type Repair: also correct migration type Nov 14, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this issue Nov 14, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this issue Nov 14, 2017
@wanglin0413
Copy link
Author

@wanglin0413 wanglin0413 commented Nov 29, 2017

I am a freshman in programming, and this is my first time to contribute to an open source project. I'm so excited to see my name in the releaseNotes. This is really a big inspiration for me. Thank u very much!

Loading

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Nov 29, 2017

@wanglin0413 No worries. It was an unusual scenario, but it made sense to include it. Thanks for the detailed report explaining your reasoning and situation. To be honest, this is often much more valuable than a pull request. Cheers!

Loading

dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants