Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(db): add vm and function to check group3_inpn #433

Merged
merged 2 commits into from
Oct 23, 2023

Conversation

mvergez
Copy link
Contributor

@mvergez mvergez commented Sep 4, 2023

Ajout de la fonction et de la vue matérialisée associée pour vérifier que la chaîne de caractère fournie est bien un groupe 3 inpn.

Ceci est nécessaire à l'ajout de la possibilité de filtrer les nomenclature selon le groupe 3 inpn. Cette PR s'inscrit dans le cadre d'une prestation pour l'Agence Régionale de la Biodiversité en île de France.

@mvergez
Copy link
Contributor Author

mvergez commented Sep 4, 2023

@amandine-sahl
Copy link
Contributor

Il manque le rafraichissement de la vm lors de la migration de taxref https://github.com/PnX-SI/TaxHub/blob/master/apptax/taxonomie/commands/utils.py#L224C6-L224C6

Updated sql function not to use vm but to select in taxref instead
@mvergez
Copy link
Contributor Author

mvergez commented Oct 16, 2023

Comme discuté avec @amandine-sahl, il est plus judicieux de créer un index sur la table taxonomie.taxref puis de faire un EXISTS sur un SELECT WHERE group3_inpn = <mon_groupe> LIMIT 1 sur la table taxonomie.taxref.

Cela fait une vue en moins à avoir à rafraichir, les performances sont équivalentes

J'ai donc actualisé la migration en conséquence

@amandine-sahl amandine-sahl merged commit 65e07c0 into PnX-SI:develop Oct 23, 2023
2 of 5 checks passed
TheoLechemia pushed a commit that referenced this pull request Oct 25, 2023
* feat(db): add vm and function to check group3_inpn

* refactor(db): replace vm by index in taxref

Updated sql function not to use vm but to select in taxref instead
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants