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

Implementing MigrationInfoProvider.getDescription() returning String longer than 200 characters #1633

Closed
gscht opened this issue May 12, 2017 · 1 comment

Comments

@gscht
Copy link

@gscht gscht commented May 12, 2017

What version of Flyway are you using?

4.2.0

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

Java API

What database are you using (type & version)?

MS SQL Server 2014

What operating system are you using?

Windows 8.1

What did you do?

(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
I created a JDBC migration implementing MigrationInfoProvider. The String getDescription() returns a String longer than 200 characters. When the migration gets applied, it works fine. The description gets abbreviated so it would fit into the metadata table.

But every subsequent call to migrate() throws an exception saying "Validate failed: Migration description mismatch for migration"

What did you expect to see?

A JDBC migration with a description longer than 200 characters should not fail after it gets applied successfully.

What did you see instead?

Validate failed: Migration description mismatch for migration

@gscht gscht changed the title Implementing MigrationInfoProvider Implementing MigrationInfoProvider.getDescription() returning String longer than 200 characters May 12, 2017
@axelfontaine axelfontaine added this to the Flyway 5.0.0 milestone May 15, 2017
@axelfontaine axelfontaine removed this from the Flyway 5.0.0 milestone Nov 29, 2017
@axelfontaine axelfontaine added this to the Flyway 5.1.0 milestone Nov 29, 2017
@axelfontaine axelfontaine removed this from the Flyway 5.1.0 milestone Dec 18, 2017
@axelfontaine axelfontaine added this to the Flyway 5.0.3 milestone Dec 18, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this issue Dec 18, 2017
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Dec 18, 2017

This has now been fixed. Validate now compares against the abbreviated version as well.

dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
…) returning String longer than 200 characters
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