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

Database migration fails partially on macOS (and other hosts with case insensitive filesystem) #6139

Open
stweil opened this issue Jul 22, 2024 · 4 comments
Labels

Comments

@stweil
Copy link
Member

stweil commented Jul 22, 2024

Describe the bug

The SQL statements in Kitodo-DataManagement/src/main/resources/db/migration/V2_105__Fixes_#3998.sql which were introduced in pull request #4412 don't work on hosts with a filesystem which is case insensitive (macOS, Windows).

An easy workaround is using an intermediate filename:

Replace

ALTER TABLE client_x_listColumn RENAME TO client_x_listcolumn;

by

ALTER TABLE client_x_listColumn RENAME TO renamed_table;
ALTER TABLE renamed_table RENAME TO client_x_listcolumn;

and similarly for all other renames.

To Reproduce
Steps to reproduce the behavior:

  1. Run migration on macOS.
  2. Check names of database tables.
  3. See error

Expected behavior
Database migration must work on macOS and Windows (and Linux with a case insensitive filesystem), too.

Release
All recent releases up to git master.

@stweil stweil added the bug label Jul 22, 2024
@stweil
Copy link
Member Author

stweil commented Jul 22, 2024

@matthias-ronge, @solth, should I prepare a PR with the suggested fix for the SQL file? Or do you have a different solution?

@stweil
Copy link
Member Author

stweil commented Jul 25, 2024

If we patch the existing migration script, this will require a flyway:repair for at least some existing installations. Is this acceptable? See related discussion for typo fixes in pull request #6149.

@henning-gerhardt
Copy link
Collaborator

My question is: why this is failing on your system? So far as I know this is working on Windows and if I remember me correct this was working on MacOS too.

@stweil
Copy link
Member Author

stweil commented Jul 25, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants