Improve coding freshness handling#799
Conversation
2e82201 to
e760320
Compare
|
Review-Hinweis / Nacharbeitswunsch: Funktional ist #688 fuer den geprueften Die Performance-Begruendung traegt aber in der aktuellen Form nicht. Ich habe den geaenderten Kernpfad
Eine Kontrollquery, die den strukturierten Scheme-Ref wirklich vorfiltert und die Unit-Datei-Zuordnung indexierbar macht, lag dagegen bei 18 ms fuer 20k Units, 89 ms fuer 100k Units und ca. 945 ms fuer 1M Units. Damit nutzt der PR die neue Denormalisierung nicht wirksam fuer Performance. Der dominante Kostenpunkt bleibt der Join ueber normalisierte Dateinamen per Ausdruck, insbesondere Bitte daher entweder den Performance-Anspruch aus PR-Beschreibung/Begruendung entfernen und die Aenderung klar als Robustheits-/Korrektheitsfix einordnen, oder den Query-/Schema-Teil nachziehen: z. B. normalisierte Unit-Datei-ID bzw. normalisierten CodingSchemeRef indexierbar speichern und |
e760320 to
3be8089
Compare
Überblick
Dieser PR adressiert Issue #688, ohne das Issue automatisch zu schließen. Ziel ist ein robusterer Korrektheitsfix für den Kodierstand nach Änderungen an
.vocs-Dateien.Änderungen
.vocs-Speichern wird direkt die Aktion „Kodierstand prüfen“ angeboten und mitrefreshCodingFreshness=1geöffnet..vocs-Uploads im Testdateien-Dialog bieten dieselbe Prüfung auch ohne Warnung an.codingSchemeRef-Metadaten instructured_data; bestehende Produktivdaten werden per Liquibase-Backfill ergänzt. Diese Zuordnung dient hier als robustere Quelle für die Freshness-Ermittlung, nicht als belegter Performance-Fix.structured_data, wenn die Metadaten vorhanden sind, und XML-Regex weiterhin als Fallback für ältere oder nicht angereicherte Daten.Validierung
.vocs-Overwrite mit Coding-Scheme-Änderung markiert Auto-Coding und manuelle Kodierung als stale/review-required..vocs-Uploads bieten in der UI direkt „Kodierstand prüfen“ mitrefreshCodingFreshness=1an..vocs-Upload zeigte die API direktv1 STALEundv2 MANUAL_REVIEW_REQUIRED; die Kodierübersicht zeigte anschließend „Kodierstand prüfen“.npx nx test backend --runInBand --testFile=apps/backend/src/app/database/services/workspace/workspace-files.service.spec.tsnpx nx lint backendnpx nx lint frontendnpx nx test frontendgit diff --checkmake dev-db-validate-changelogHinweis zur Performance
Dieser PR sollte nicht als nachgewiesener Performance-Fix bewertet werden. Die aktuelle Query-Form profitiert bei sehr großen Datenmengen nicht ausreichend von der Denormalisierung, weil der Join zwischen Units und Unit-Dateien weiterhin über normalisierte Dateinamen per Ausdruck erfolgt. Ein echter Performance-Fix sollte die Zuordnung zusätzlich indexierbar machen und die Suche zuerst über den strukturierten Scheme-Ref einschränken.
Refs #688