-
Notifications
You must be signed in to change notification settings - Fork 1
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
Possible changes to Climsoft 4 data model #154
Comments
Currently, we know which observation is the "current best value" because it is stored in @maxwellfundi - If we know which record is the current best value, is there any reason why we need to record the order in which the other previous versions of the observation were recorded? |
@isedwards I thought that the best version is not necessarily always in the |
Yes, you're right. That is how it currently works. |
Do we continue with this or propose something different? |
We should continue to use |
Ok. Does this mean that adding any new fields to these tables would cause problems for existing users? |
We may be able to add fields without causing any problems. The new code will need to be able to cope with observations that do not have data in the new fields. When the new field is first created, existing rows will either be null, or they can have a default that we specify. We should keep in mind that an organisation could have other machines that still connect to the database using instances of v4.1.x as well as our new 4.2 (even if this is only for a couple of days, but it's easy to imagine that it could be longer). So we need to understand how this would create inconsistencies and be able to handle legacy observation when we encounter them. We can't detect and fix all inconsistencies on each start up because it would be too slow. If 4.1.x and 4.2 cannot coexist then we'll have to consider moving to version 5. Alternatively we'll have to work with the places that we know have installations to make sure everything is updated simultaneously (but that would be bad practice because other people may be using the software unsupported). |
The text was updated successfully, but these errors were encountered: