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

Update H2Schema to work with DATABASE_TO_UPPER=false #2325

Merged
merged 2 commits into from Apr 9, 2019

Conversation

@josephlbarnett
Copy link
Contributor

@josephlbarnett josephlbarnett commented Mar 12, 2019

With H2 1.4.198 (and 1.4.199) and DATABASE_TO_UPPER=false, flyway fails being
unable to find the (lowercase) schemata table and related columns.

Updating to use uppercase versions of the same makes things work.
With H2 1.4.197, this change was not necessary for some reason though?

With H2 1.4.198 and DATABASE_TO_UPPER=false, flyway fails being
unable to find the (lowercase) schemata table and related columns.

Updating to use uppercase versions of the same makes things work.
With H2 1.4.197, this change was not necessary for some reason though?
@CLAassistant
Copy link

@CLAassistant CLAassistant commented Mar 12, 2019

CLA assistant check
All committers have signed the CLA.

@josephlbarnett
Copy link
Contributor Author

@josephlbarnett josephlbarnett commented Mar 19, 2019

Need anything further from me to move this forward?

@axelfontaine axelfontaine merged commit 873285c into flyway:master Apr 9, 2019
1 of 2 checks passed
axelfontaine pushed a commit to flyway/flywaydb.org that referenced this issue Apr 9, 2019
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Apr 9, 2019

Many thanks. Merged. I have also added you to the hall of fame page.

@axelfontaine axelfontaine removed this from the Flyway 6.0.0 milestone May 22, 2019
@axelfontaine axelfontaine added this to the Flyway 6.0.0-beta2 milestone May 22, 2019
dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
Update H2Schema to work with DATABASE_TO_UPPER=false
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

3 participants