FlexMeasures / flexmeasures / 24003009626

78%

main: 81%

Build Type

Pull #1985

github

Committed by web-flow

Commit Message

fix(migration): handle duplicate DataSource rows and simplify downgrade

Upgrade: before updating attributes_hash values, detect groups of DataSource
rows that share the same normalised unique key (after JSONB sorting). For each
such conflict group the newest row (highest id) is kept clean; older duplicates
get a synthetic {"flexmeasures-hash-conflict": N} attribute so their hash stays
unique -- avoiding a UniqueConstraint violation without touching timed_belief.

Downgrade: removed the hash-recomputation loop. PostgreSQL JSONB stores object
keys in alphabetical order, so json.dumps(attrs) and
json.dumps(attrs, sort_keys=True) produce the same string for any JSONB-round-
tripped attributes -- making the old loop a no-op. The downgrade is now an
intentional pass with an explanatory docstring.

Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/c6f63d53-58a6-4dc3-9710-7e5846770626

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

Pull Request Pull Request #1985: Add "Create forecast" button to sensor page

Coverage Stats

22 of 24 new or added lines in 4 files covered. (91.67%)

3 existing lines in 1 file now uncovered.

11597 of 14897 relevant lines covered (77.85%)

0.78 hits per line