search

happy-hamster / backend

RESTful Spring Boot API
https://api.happyhamster.org
Apache License 2.0
2 stars 0 forks source link

TINF-251: Anpassen der Datentypen im Datenbankmodell #129

Closed Mavv3006 closed 4 years ago

Mavv3006 commented 4 years ago

Ich habe mich darauf konzentriert die String Felder zu kürzen.

Wenn ich noch weitere Wünsche habt, bitte hier kommentieren.

DavidSchmidt00 commented 4 years ago

Bringt das Performance Vorteile? ^^

Mavv3006 commented 4 years ago

Bringt das Performance Vorteile? ^^

Habe ich bisher nicht gemessen. Weiß ich nicht. Ist aber nötig gewesen, weil viele Felder Platz für einen String mit 255 Zeichen hatte. Das ist nicht nötig.

r-franzke commented 4 years ago

Welche Auswirkung hat das auf die Datenbank? Kann man die einfach weiter verwenden?

Fabsi23 commented 4 years ago

Sollten nicht auch noch die Datentypen der Felder angepasst werden? Warum sollte beispielsweise POSTCODE in Address ein String sein, wenn es lediglich numerische Postleitzahlen gibt?

jstsddg commented 4 years ago

@DavidSchmidt00 Das müsste eigentlich schneller sein, wenn die Strings kleiner von der Länge sind, dann können die direkt in der Tabelle liegen und müssen nicht in eine andere Datei ausgelagert werden. (Ich weiß nicht mehr wo ich das gelernt habe, aber bin mir recht sicher, dass es so ist.)

Und wenn wir die Optimierung von @Fabsi23 noch nehmen, dann auf jeden Fall.

Mavv3006 commented 4 years ago

Sollten nicht auch noch die Datentypen der Felder angepasst werden? Warum sollte beispielsweise POSTCODE in Address ein String sein, wenn es lediglich numerische Postleitzahlen gibt?

In DE gibt es nur numerische Postleitzahlen. In Canada zum Beispiel aber nicht.

Fabsi23 commented 4 years ago

Laut Wikipedia sollen Postleitzahlen weltweit maximal 10 Zeichen haben. Mit den Strings hattest du Recht. Änderst du das noch von 20 auf 10 Zeichen Länge?

Und benötigen wir 50 Zeichen für eine Hausnummer? Selbst wenn mal eine 127/11 rauskommt sind wir davon noch weit entfernt.

Mavv3006 commented 4 years ago

Laut Wikipedia sollen Postleitzahlen weltweit maximal 10 Zeichen haben. Mit den Strings hattest du Recht. Änderst du das noch von 20 auf 10 Zeichen Länge?

Und benötigen wir 50 Zeichen für eine Hausnummer? Selbst wenn mal eine 127/11 rauskommt sind wir davon noch weit entfernt.

Aktuell ist die längste Hausnummer 35 Zeichen lang: 57;59;61;63;65;67;69;71;73;75;77;79. Ich wollte ein wenig Spielraum lassen. Aber guter Einwand bzgl. der Postleitzahlen. Werde ich mir anschauen und gegebenenfalls ändern. 👍

Mavv3006 commented 4 years ago

Beim Importieren müssten dann die importierten Werte (falls nötig) zugeschnitten werden, damit der Import nicht crasht :)

@MrKraboom Die Werte werden nicht zugeschnitten, weil für alle Werte genügend Platz ist. Andernfalls müssen die Felder wieder länger gemacht werden.

jstsddg commented 4 years ago

Und was passiert wenn OSM die unterliegenden Daten ändert? Dann bricht uns der Import zusammen und müssen das backend Hotfixen, wenn ein paar Daten zu lang sind. Das ist doch auch nicht Sinn der Sache, oder?

DavidSchmidt00 commented 4 years ago

Naund? Hotfixes sind toll

jstsddg commented 4 years ago

Was ist das denn für ein @SAKPaaS/product-owner o_O Mehr arbeit! :partying_face:

DavidSchmidt00 commented 4 years ago

das wär viel zu viel aufwand denke ich ma^^ für ne sache die evtl. später mal eintreten könnte

Mavv3006 commented 4 years ago

Ich konnte den Fehler bei mir nicht repoduzieren.