Microsoft/bcquality
1
0
Fork 0
mirror of https://github.com/microsoft/BCQuality.git synced 2026-08-06 09:26:52 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions
bcquality/microsoft/knowledge/upgrade/check-only-triggers-do-not-migrate-data.md
Jesper Schulz-Wedde e0ebdd35c7
Add lifecycle error and privacy knowledge (#99) 
* Add lifecycle error and privacy knowledge

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 95c06ad8-377d-4faa-8d07-06300b1c81ec

* Fix lifecycle privacy review findings

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: a26cd6d6-ff49-433e-bc53-f645c455ebdd

* Refine lifecycle privacy retrieval

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 95c06ad8-377d-4faa-8d07-06300b1c81ec

* Make review gates explicit

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 95c06ad8-377d-4faa-8d07-06300b1c81ec

---------

Co-authored-by: Jesper Schulz-Wedde <jesper.schulzwedde@microsoft.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
2026-07-14 12:52:56 +02:00

1.2 KiB
Raw Permalink Blame History

bc-version domain keywords technologies countries application-area
all
upgrade
on-check-preconditions
on-validate-upgrade
on-upgrade
read-only-check
data-migration
al
w1
all

Upgrade check triggers do not migrate data

Description

OnCheckPreconditionsPerCompany/PerDatabase run before the upgrade to verify that it can start. OnValidateUpgradePerCompany/PerDatabase run after upgrade logic to verify that it succeeded. Treat both phases as read-only checks. The OnUpgradePerCompany/PerDatabase phase is where the platform expects actual data transformation.

Best Practice

Have check triggers call query-only helpers that raise an error when an invariant fails. Put every Insert, Modify, Delete, Rename, DataTransfer, and other migration write behind helpers called from the matching OnUpgrade... trigger.

See sample: check-only-triggers-do-not-migrate-data.good.al.

Anti Pattern

Repairing data in OnCheckPreconditions... or finishing migration in OnValidateUpgrade.... Those writes blur the phase contract and make a check alter the state it is supposed to assess.

See sample: check-only-triggers-do-not-migrate-data.bad.al.