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

🗃️ add database layout for IFOPT #2413

Merged
merged 7 commits into from
Mar 15, 2024
Merged

Conversation

MrKrisKrisu
Copy link
Member

see #2411

@jeyemwey
Copy link
Contributor

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 : in between.

@HerrLevin
Copy link
Member

@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.

@jeyemwey
Copy link
Contributor

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.

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

@MrKrisKrisu
Copy link
Member Author

It's not necessary per say

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)
@MrKrisKrisu MrKrisKrisu merged commit 28bb3bd into develop Mar 15, 2024
10 checks passed
@MrKrisKrisu MrKrisKrisu deleted the dev-ks/2411/ifopt branch March 15, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants