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

Fix: add indexes to vm_observations_mailles #393

Closed
wants to merge 2 commits into from
Closed

Conversation

jpm-cbna
Copy link
Contributor

Improve atlas.vm_observations_mailles with new indexes.

@jpm-cbna jpm-cbna added the bug label Mar 23, 2022
@camillemonchicourt
Copy link
Member

A mettre dans un petit fichier d'update SQL aussi je pense pour ceux qui auraient déjà la 1.5.0 installée.

@jpm-cbna
Copy link
Contributor Author

A mettre dans un petit fichier d'update SQL aussi je pense pour ceux qui auraient déjà la 1.5.0 installée.

Oui, bien vu ! Mais par contre, je voie plusieurs solutions pour le faire :

  • je rajoute juste un nouveau fichier "update_vm_index.sql" dans le dossier data/update/ qui sera ensuite fusionner dans le future update_1.5.1to1.5.2.sql lors de la future release
  • je mets à jour le fichier data/update/update_vm_observations.sql et on rejoue la mise en place de toutes les VM. Sachant que ce fichier ne semble pas maintenu.
  • je mets en place les 2 solutions précédentes

Concernant le fichier data/update/update_vm_observations.sql, il me semble qu'il devrait être remplacé par un script Bash. Cela permettrait de rejouer les scripts présents dans le dossier data/atlas/ pour éviter de dupliquer le code SQL. Il pourrait aussi se servir correctement des variables définies dans settings.ini plutôt que d'avoir l'utilisateur geonatatlas en dur dans le code.

@camillemonchicourt
Copy link
Member

En effet quand on met à jour GeoNature-atlas, si possible on ne demande pas de récréer toutes les VM car souvent chacun les a adapté à son contexte.
Donc une fichier SQL avec les évolution depuis la dernière version est idéal.

On avait déjà évoqué le fait que garder en doublon partiel ce fichier data/update/update_vm_observations.sql était pas une bonne idée, pas à jour car en partie en doublon, donc à dégager selon moi (#346 (comment)).

@jpm-cbna
Copy link
Contributor Author

jpm-cbna commented Apr 7, 2022

@TheoLechemia Je viens rajouter le fichier "update_vm_index.sql" dans le dossier data/update/. Il pourra servir à constituer le fichier de migration update_1.5.1to1.5.2.sql.

@jpm-cbna jpm-cbna closed this May 20, 2024
@jpm-cbna
Copy link
Contributor Author

Les changements réalisés récemment sur la VM vm_observations_mailles et intégré à la branche develop rende cette PR inutile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants