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

Fix Update 0439 #241

Merged
merged 2 commits into from
Jun 14, 2024
Merged

Conversation

JohannMaierhofer
Copy link

Wie in #223 festgestellt, funktioniert der Code dort nicht mit einer MariaDB.
Da ich bei einer MySQL Datenbank keinen leichten Weg gefunden habe um zu prüfen ob der Foreign Key schon existiert schlage ich eine pragmatische Lösung vor.
Bei MySQL erzeuge ich einfach den Foreign Key auch wenn er schon existiert. In Java fange ich die Exception dann ab und werfe keine Application Exception. Dann läuft die SW einfach weiter ohne Probleme.

Ich habe mir dazu eine MariaDB eingerichtet und damit getestet.

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Jun 11, 2024
@dippeal
Copy link
Member

dippeal commented Jun 12, 2024

Willst du nicht lieber in der Methode createForeignKeyIfNotExistsNocheck die Lösung unterbringen. Mit der neuen Methode executeNoCheck verwirfst du ja auch Fehlermeldungen der H2 DBs bei diesem 0439 Update.

@JohannMaierhofer
Copy link
Author

Ja, kann ich machen.

@JohannMaierhofer
Copy link
Author

Habe die Lösung jetzt in createForeignKeyIfNotExistsNocheck wie vorgeschlagen eingebaut.

@dippeal dippeal merged commit 1751159 into openjverein:master Jun 14, 2024
@JohannMaierhofer JohannMaierhofer deleted the fix_Update0439 branch June 14, 2024 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix This will fix a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants