veeso / ic-dbms / 21727735177

95%

main: 96%

Build Type

push

github

Committed by web-flow

Commit Message

perf(canister): implement in-place update instead of delete+insert (#37)

Replace the delete-then-insert update strategy with a proper in-place
update approach in TableRegistry, improving performance by avoiding
unnecessary memory reallocation when record size is unchanged.

- Add TableRegistry::update with two-path strategy: in-place overwrite
  for same-size records, delete+reinsert for size changes
- Add UpdateIntegrityValidator that allows keeping the same PK during
  updates, unlike InsertIntegrityValidator which rejects any existing PK
- Add validate_update to DatabaseSchema trait and macro generation
- Extract shared validation logic (column, FK, non-nullable checks)
  into integrity::common module to eliminate duplication
- Cascade PK changes to referencing tables via
  update_pk_referencing_updated_table
- Remove DeleteBehavior::Break variant (workaround for old strategy)
- Extract sanitize_values helper from insert/update flows
- Add tests for multi-record updates, PK conflict e2e, FK cascade
  in transactions, and non-nullable field validation

Closes #11

Run Details

485 of 493 new or added lines in 5 files covered. (98.38%)

2 existing lines in 1 file now uncovered.

9765 of 10211 relevant lines covered (95.63%)

735272.28 hits per line