Add PK for taxonomie.import_taxref #359
Conversation
La table `taxonomie.import_taxref` est créée et peuplée sans clé primaire. Donc sans index. Lors de l'import du nouveau taxref de `taxonomie.import_taxref` vers `taxonomie.taxref`, la requête ne trouve pas d'index sur le cd_nom de `taxonomie.import_taxref`. Elle est donc extrêmement longue. En ajoutant la PK, la mise à jour de taxref est incomparablement plus rapide (testé). Syntaxe de la modif non testée. A vérifier donc.
Codecov ReportBase: 51.64% // Head: 51.58% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #359 +/- ##
===========================================
- Coverage 51.64% 51.58% -0.06%
===========================================
Files 31 31
Lines 2012 2014 +2
===========================================
Hits 1039 1039
- Misses 973 975 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
Effectivement je pense que c'est plus pertinent de créer la clé primaire au moment de la création de la table |
|
Oui il me semble aussi. Le mieux est de rejeter cette PR et de faire la modif dans le fichier qui crée la table. |
PR par @gildeluermoz
La table
taxonomie.import_taxrefest créée et peuplée sans clé primaire. Donc sans index. Lors de l'import du nouveau taxref detaxonomie.import_taxrefverstaxonomie.taxref, la requête ne trouve pas d'index sur le cd_nom detaxonomie.import_taxref. Elle est donc extrêmement longue. En ajoutant la PK, la mise à jour de taxref est incomparablement plus rapide (testé). Syntaxe de la modif non testée. A vérifier donc.Peut-être qu'il serait plus simple de créer cette PK dans le fichier https://github.com/PnX-SI/TaxHub/tree/master/apptax/taxonomie/commands/migrate_to_v15/data/0_taxrefv15_import_data.sql