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

Issue 598 fix #609

Merged
merged 2 commits into from Dec 9, 2013
Merged

Issue 598 fix #609

merged 2 commits into from Dec 9, 2013

Conversation

@bigslack
Copy link

@bigslack bigslack commented Sep 20, 2013

Patch to resolve most of issue #598. In particular cases 2 & 3 where the schema is set by .setSchema() and no schema is present in the datasource url.

Adam Green added 2 commits Sep 20, 2013
…ck to originalSchema. Should now properly handle setups where the datasource url does not have a schema defined but instead .setSchema() was used.
axelfontaine added a commit that referenced this pull request Dec 9, 2013
Issue 598 fix
@axelfontaine axelfontaine merged commit 0c6b3d6 into flyway:master Dec 9, 2013
1 check failed
1 check failed
default The Travis CI build could not complete due to an error
Details
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Dec 9, 2013

Hi Adam,

thanks for the pull request. I merged it, added a test, found out it broke PostgreSQL support and eventually rewrote it :-)

This should be easier to check in the future though, now that the Travis CI also runs the MySQL and PostgreSQL tests.

Cheers
Axel

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.