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

Flyway doesn't recognize Sybase ASE 12.5 #1483

Closed
ogerardin opened this issue Dec 8, 2016 · 3 comments
Closed

Flyway doesn't recognize Sybase ASE 12.5 #1483

ogerardin opened this issue Dec 8, 2016 · 3 comments
Labels
Milestone

Comments

@ogerardin
Copy link

@ogerardin ogerardin commented Dec 8, 2016

It seems version 12.5 of Sybase ASE identifies itself as "sql server 12.50", while DbSupportFactory.createDbSupport expects something that starts with "ASE" or "Adaptive.

What version of Flyway are you using?

4.0.3

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)?

Sybase ASE 12.5

What operating system are you using?

Windows 7

What did you do?

(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
flyway -X -url=jdbc:jtds:sybase://:4100/ baseline

What did you expect to see?

Not an error

What did you see instead?

ERROR: Unsupported Database: sql server 12.50

@ogerardin
Copy link
Author

@ogerardin ogerardin commented Dec 8, 2016

A simple solution to this would be to provide an option to bypass DbSupport factory, something like
flyway.dbSupport=org.flywaydb.core.internal.dbsupport.sybase.ase.SybaseASEDbSupport

@ogerardin
Copy link
Author

@ogerardin ogerardin commented Dec 8, 2016

I created a pull request for this issue, the ball is in your court now...

@axelfontaine axelfontaine added this to the Flyway 4.1 milestone Jan 31, 2017
axelfontaine added a commit to flyway/flywaydb.org that referenced this issue Jan 31, 2017
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Jan 31, 2017

Thanks for sharing the exact string. Support has now been added.

@axelfontaine axelfontaine added the t: bug label Jan 31, 2017
pradheeps pushed a commit to pradheeps/flyway that referenced this issue Mar 7, 2017
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
You can’t perform that action at this time.