[TECH] Migrer la colonne Answer.id de INTEGER en BIG INTEGER (Partie 3)

[yahyayouness]

🦄 Problème

Voir Partie 1 #3357
Voir Partie 2 #4127

🤖 Solution

Créer les

• contraintes d'intégrités
• index

Pour les deux tables temopraires:

• answers_bigint
• knowledge-elements_bigint.

🌈 Remarques

Résultat sur une environnement iso prod: 6h pour la création des contraintes et d'index 15h lors du dernier test.

Test 1
,"pid":24,"hostname":"pix-int-to-bigint-test-one-off-1115","msg":"Start adding constraints and indexes to answers ke bigint tables"}
2022-02-18 11:55:36.280651704 +0100 CET [one-off-1115] {"level":30,"time":1645181736280,"pid":24,"hostname":"pix-int-to-bigint-test-one-off-1115","msg":"Start script... "}
2022-02-18 11:55:36.370391091 +0100 CET [one-off-1115] {"level":30,"time":1645181736369,"pid":24,"hostname":"pix-int-to-bigint-test-one-off-1115","msg":"number of rows answer table 707264980"}
2022-02-18 11:55:36.370397041 +0100 CET [one-off-1115] {"level":30,"time":1645181736369,"pid":24,"hostname":"pix-int-to-bigint-test-one-off-1115","msg":"create indexes and constraints for answers_bigint table"}
2022-02-18 15:40:47.705346425 +0100 CET [one-off-1115] {"level":30,"time":1645195247704,"pid":24,"hostname":"pix-int-to-bigint-test-one-off-1115","msg":"\nEnd script: copy done successfully."}

Test 2
Start script...
2022-02-09 17:17:03.320709810 +0100 CET [one-off-4015] Start adding constraints to answers ke bigint tables
2022-02-09 17:17:03.632666565 +0100 CET [one-off-4015] number of rows answer table 686892528
2022-02-09 17:17:03.632677390 +0100 CET [one-off-4015] create indexes and constraints for answers_bigint table
2022-02-09 18:06:54.436501877 +0100 CET [one-off-4015] create indexes and constraints for knowledge-elements_bigint table
2022-02-09 23:21:57.814662289 +0100 CET [one-off-4015] Call to addConstraintsAndIndexToAnswersKeBigintTemporaryTables took 21894493.60166931 milliseconds
2022-02-09 23:21:57.814666098 +0100 CET [one-off-4015] Finish add constraints

💯 Pour tester

En local:

1/ Lancer le script avec l'instruction:

LOG_LEVEL=debug node /scripts/bigint/answer/add-constraints-and-indexes-to-answers-ke-temporary-tables-with-bigint.js

2/ vérifier que toutes les contraintes et indexes des tables answer_bigint et knowledge-elements_bigint ont été créés avec succès.

En RA:

1/ Lancer le script avec l'instruction:

LOG_LEVEL=debug node /scripts/bigint/answer/add-constraints-and-indexes-to-answers-ke-temporary-tables-with-bigint.js

2/ vérifier que toutes les contraintes et indexes des tables answer et KE ont été créés avec succès.

table answer vers answer_bigint
table knowledge-elements vers knowledge-elements_bigint

Env iso production / Application pix-int-to-bigint

voir log dans la description dans la PR. (Done)

[pix-service]

I'm deploying this PR to these urls:

• App (.fr): https://app-pr4130.review.pix.fr
• App (.org): https://app-pr4130.review.pix.org
• Orga: https://orga-pr4130.review.pix.fr
• Certif: https://certif-pr4130.review.pix.fr
• Admin: https://admin-pr4130.review.pix.fr
• API: https://api-pr4130.review.pix.fr/api/

Please check it out!

[octo-topi]

Il y a une contrainte NOT NULLABLE en trop (j'ai push un commit)
J'ai aussi rajouté un test automatisé
Je propose aussi de mettre le scope de la transaction et de supprimer le commit explicite (voir #4127 (comment))
Si ces trois points sont corrigés (et un test relancé sur les données iso-prod), c'est ✔️ pour moi

[octo-topi]

Bon pour moi ! Mergeable dès que le test avec des données iso-production est rééexécuté et ne montre pas de régressions.