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

Failed SQL migration error reporting improvement #649

Merged
merged 2 commits into from Nov 29, 2013

Conversation

@bpg
Copy link

@bpg bpg commented Nov 24, 2013

This pull request fixes issues #575, #587, #647.
A new exception class FlywaySqlScriptException (subclass of FlywayException) was introduced in order to explicitly capture information related to error (script line, statement, jdbc exception), so it can be used in client code to implement application-specific exception handling if required.

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Nov 28, 2013

Thanks for the fix. I can't merge it until the build passes though.

Cheers
Axel

@bpg
Copy link
Author

@bpg bpg commented Nov 28, 2013

Thanks for the heads up, I missed that Travis CI report...
Pavel.

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Nov 28, 2013

Don't put in too much effort yet. I believe the issue is on their side: travis-ci/travis-ci#1689

axelfontaine added a commit that referenced this pull request Nov 29, 2013
Failed SQL migration error reporting improvement
@axelfontaine axelfontaine merged commit 421a156 into flyway:master Nov 29, 2013
1 check failed
1 check failed
default The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.