-
Notifications
You must be signed in to change notification settings - Fork 47
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
🗃️ add database layout for IFOPT #2413
Conversation
Why is it necessary to have 5 fields in the database for this? IMO it would be sufficient to keep it in one column with |
@jeyemwey It's not necessary per say, but you can leverage the relational characteristics of the individual segments. That way, we can group stations based on segment a, b and c to use another API or stuff like that. Doing that with LIKE will be way less efficient. |
database/migrations/2024_03_10_211526_add_ifopt_to_stations.php
Outdated
Show resolved
Hide resolved
database/migrations/2024_03_10_211526_add_ifopt_to_stations.php
Outdated
Show resolved
Hide resolved
database/migrations/2024_03_10_211526_add_ifopt_to_stations.php
Outdated
Show resolved
Hide resolved
database/migrations/2024_03_10_211526_add_ifopt_to_stations.php
Outdated
Show resolved
Hide resolved
database/migrations/2024_03_10_211526_add_ifopt_to_stations.php
Outdated
Show resolved
Hide resolved
Co-authored-by: Levin Herr <[email protected]>
Co-authored-by: Levin Herr <[email protected]>
Co-authored-by: Levin Herr <[email protected]>
Co-authored-by: Levin Herr <[email protected]>
Jetzt wo ich drüber nachdenke, kann ich die Chancen sehen. Allein die Chance, Country-Informationen darzustellen, und zwar abseits von Ziffern in einer IBNR, ist ja schon nice. :3 |
In the context of database normalization, I would argue that this is the most sensible approach. Since each attribute provides its own context, it is also easier to select - and I also see improved performance if we later want to search by country or municipal unit, for example. |
Levin [15.03.24 14:00] Ich glaub, ich hab mich beim Tippen auf dem Handy dumm angestellt oder so. Das hier wäre der richtige Change gewesen. Idk, warum ich das doppelt habe. #2413 (comment)
see #2411