DorkasV / jira-to-github-issues

0 stars 0 forks source link

Databasen har foreignkeys der ikke er trusted #77

Open sync-by-unito[bot] opened 4 months ago

sync-by-unito[bot] commented 4 months ago

Info til Cowi.

Databasen har foreignkeys der ikke er trusted. Sandsyndligvis har de været deaktiveret på et tidspunkt, men ikke efterfølgende trusted med ”WITH CHECK CONSTRAINT”. Uden dette vil SQL serveren ikke bruge foreign keys, da den ikke har nogen garanti for de er valide.

Hvis Cowi har deaktiveret foreignkeys, så skal I huske at aktivere dem igen med ”WITH CHECK CONSTRAINT”.

Script til retellse af dette er vedlagt.

Cowi må gerne lukke dette issue efter I har forholdt jer til dette.

┆Attachments: Untrusted Foreign Keys - med cursor.sql

sync-by-unito[bot] commented 4 months ago

➤ Daniel Balint Szucs commented:

Hej Peter Riis Hvilke miljøer/regioner skal det køres på? Hvem har skrevet det, og hvor blev det testet?

sync-by-unito[bot] commented 4 months ago

➤ Peter Riis commented:

Jeg har selv lavet det og kørt det på vores TEST. Kører det også snart på PROD.

https://www.brentozar.com/blitz/foreign-key-trusted/ ( https://www.brentozar.com/blitz/foreign-key-trusted/|smart-link )

Det er mest til information om at hvis I har deaktiveret foreignkeys, så skal I huske at aktivere dem igen med ”WITH CHECK CONSTRAINT”.

Scriptet retter det på alle databaser på serveren.

I kan jo overveje at kører det hos jer selv.

sync-by-unito[bot] commented 4 months ago

➤ Emil Sahin commented:

Hej Peter Riis

Er denne opgave ikke noget vi skal tage op til næste PG-møde? 🙂

sync-by-unito[bot] commented 4 months ago

➤ Peter Riis commented:

Det kan vi sagtens. Vi talte også om det på workshoppen

sync-by-unito[bot] commented 4 months ago

➤ Hans Christian Ries commented:

Region syd FK:

keyname [dbo].[r_matrikel_liste].[FK_r_matrikel_liste_e_matrikel1] [dbo].[r_matrikel_ejendom].[FK_r_matrikel_ejendom_e_ejendom1] [dbo].[r_matrikel_ejendom].[FK_r_matrikel_ejendom_e_matrikel1] [dbo].[r_haandtering_liste].[FK_r_haandtering_liste_e_forurening_haandtering1] [dbo].[r_haandtering_aktoer_liste].[FK_r_haandtering_aktoer_liste_e_aktoer] [dbo].[r_haandtering_aktoer_liste].[FK_r_haandtering_aktoer_liste_e_forurening_haandtering] [dbo].[r_gis_matrikel].[FK_r_gis_matrikel_e_matrikel] [dbo].[r_forureningsmedie_stoffer].[FK_r_forureningsmedie_stoffer_e_forureningsmedie1] [dbo].[r_rolle_rettighed].[FK_r_rolle_rettighed_e_rolle] [dbo].[r_forurening_sagsbehandler].[FK_r_forurening_sagsbehandler_e_forurening_version] [dbo].[r_forurening_matrikel].[FK_r_forurening_matrikel_e_forurening_version] [dbo].[r_forurening_matrikel].[FK_r_forurening_matrikel_e_matrikel1] [dbo].[r_forurening_liste].[FK_r_forurening_liste_e_forurening_version] [dbo].[r_alarm_liste].[FK_r_alarm_liste_e_liste] [dbo].[r_forurening_identifikation].[FK_r_forurening_identifikation_e_forurening_version] [dbo].[e_abonnent].[FK_e_abonnent_e_forurening_master] [dbo].[r_forurening_adresse].[FK_r_forurening_adresse_e_forurening_version] [dbo].[r_fase_liste].[FK_r_fase_liste_e_fase] [dbo].[e_liste_liste].[FK_e_liste_liste_e_liste_liste_definition] [dbo].[r_bort_olie_kemi_liste].[FK_r_bort_olie_kemi_liste_e_bort_olie_kemi1] [dbo].[r_ejendom_liste].[FK_r_ejendom_liste_e_ejendom1] [dbo].[r_dokument_liste].[FK_r_dokument_liste_e_dokument1] [dbo].[r_ejendom_adresse].[FK_r_ejendom_adresse_e_adresse1] [dbo].[r_ejendom_adresse].[FK_r_ejendom_adresse_e_ejendom1] [dbo].[r_forureningskilde_flade].[FK_r_forureningskilde_flade_e_flade1] [dbo].[r_bruger_rolle].[FK_r_bruger_rolle_e_rolle] [dbo].[r_alarm_liste].[FK_r_alarm_liste_e_alarm] [dbo].[r_aktoer_type].[FK_r_aktoer_type_e_aktoer] [dbo].[r_adresse_liste].[FK_r_adresse_liste_e_adresse1] [dbo].[e_projekt].[FK_e_projekt_e_forurening_version] [dbo].[e_historik].[FK_e_historik_e_forurening_master] [dbo].[r_validering_regel_tilknytning].[FK_r_validering_regel_tilknytning_e_validering_regel] [dbo].[r_forureningskilde_liste].[FK_r_forureningskilde_liste_e_forureningskilde1] [dbo].[e_forurening_version].[FK_e_forurening_version_e_forurening_master] [dbo].[r_forureningskilde_flade].[FK_r_forureningskilde_flade_e_forureningskilde1] [dbo].[e_aktoer].[FK_e_aktoer_e_adresse] [dbo].[r_aktoer_ejendom].[FK_r_aktoer_ejendom_e_aktoer] [dbo].[e_aktivitet].[FK_e_aktivitet_e_forurening_version] [dbo].[r_aktoer_ejendom].[FK_r_aktoer_ejendom_e_ejendom1] [dbo].[s_jordtransport_detaljer].[FK_s_jordtransport_detaljer_s_jordtransport1] [dbo].[soegning_kriterie].[FK_soegning_kriterie_soegning] [dbo].[s_rens_metode].[FK_s_rens_metode_e_forurening_haandtering1] [dbo].[r_forurening_haandtering_medier].[FK_r_forurening_haandtering_medier_e_forurening_haandtering1] [dbo].[s_oevrige].[FK_s_oevrige_e_forureningskilde1] [dbo].[r_forurening_haandtering_medier].[FK_r_forurening_haandtering_medier_e_forureningsmedie1] [dbo].[s_jordtransport].[FK_s_jordtransport_e_forurening_haandtering1] [dbo].[s_deponi].[FK_s_deponi_e_forureningskilde1] [dbo].[r_projekt_liste].[FK_r_projekt_liste_e_projekt]

sync-by-unito[bot] commented 4 months ago

➤ Peter Riis commented:

Tom Birch Hansen og Hans Christian Ries undersøger dette issue i deres miljøer, efter aftale på mødet den 27-04-2023.

sync-by-unito[bot] commented 4 months ago

➤ Hans Christian Ries commented:

Jeg fik en fejl : The ALTER TABLE statement conflicted with the FOREIGN KEY constraint "FK_r_alarm_liste_e_alarm". The conflict occurred in database "JAR_DRIFT_DB_2015", table "dbo.e_alarm", column 'pkid'.