Skip to content
This repository has been archived by the owner on Aug 22, 2024. It is now read-only.

Remove double datamodel definition #17

Open
olivierdalang opened this issue Mar 16, 2023 · 1 comment
Open

Remove double datamodel definition #17

olivierdalang opened this issue Mar 16, 2023 · 1 comment
Labels
sustainability Improvement of the code with the aim of making it easier to maintain / reduce the technical debt

Comments

@olivierdalang
Copy link
Contributor

Currently, datamodel is defined both as init scripts and through deltas, leading to a duplicate definition that often gets out of sync.

We should remove the datamodel definition, and only keep deltas. This requires adapting PUM so it is able to initialize an empty database and adding an initial 1.0.0 delta.

Note: we can keep an SQL representing the full datamodel as a build artifact for documentation purposes.

@olivierdalang olivierdalang added the sustainability Improvement of the code with the aim of making it easier to maintain / reduce the technical debt label Mar 16, 2023
@olivierdalang olivierdalang added this to the Wasterwater 2023.1 milestone Mar 16, 2023
@sjib
Copy link
Contributor

sjib commented Mar 17, 2023

Note: we can keep an SQL representing the full datamodel as a build artifact for documentation purposes.

I would still provide this as a mandatory requirement - if you just want to setup the datamodel seperately somewhere (testing or whatever, where you do not have the pum tools available to go with all the deltas)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
sustainability Improvement of the code with the aim of making it easier to maintain / reduce the technical debt
Projects
None yet
Development

No branches or pull requests

2 participants