-
Notifications
You must be signed in to change notification settings - Fork 10
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
Elaboration of MongoDB tables - Database structure. #1019
Comments
Sorry, I believe it is a very bad idea to elaborate on the document collections: the last thing we want is for people to start depending on these. We need to reserve the freedom to change these as we see fit in order to improve performance or for whatever other reason. Your concern on the empty collections is however legit: these are remains of a previous server & storage architecture and should ideally be removed. |
Hello @rorlic, I agree with your idea. From my perspective, we need to have it documented somewhere before the end of project. |
@rorlic i think it's due to Mongock migrations.. Needs to be looked at. But I follow you on documenting these tables. We keep track of our DB changes via Mongock which gives an explanation. If we start writing static code, we are doomed to have documentation errors |
TO DO: delete redundant tables after clearing in mongock steps. |
The application (develop branch) currently only creates the tables that are used. I know that above was an issue in the past but I was not able to reproduce it anymore with the latest server and mongo. Whether or not we should document the usage of the other database collections can be part of #346. Since we may migrate to PostgreSQL in the near future, it may be best to wait with this as well. |
LDES Server - Snapshot 2.4.0
In MongoDB, 14 tables are created once launched the LDES Server.
Some of them remain empty.
e.g:
It might be a good idea to elaborate on the usage of each table in the doc, for the future open-source community and remove the redundant ones.
Thanks.
The text was updated successfully, but these errors were encountered: