perf: Amélioration de la performance de la requête dans getLasDiscoveries #455
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Before, queries's duration to get 6 last discoveries : 7sec New query duration: 0.34 sec (analyze summary from DBEAVER based on atlas06 and atlas13) atlas13 : https://hop-levivant.fr/ (13) atlas06 : https://www.biodiv06-lpo.org/ Reviewed-by: andriac (Natural Solutions)
|
OK merci pour ce retour, intéressant en effet. PS : En passant, je ne connaissais pas ces portails. Mais quand des nouveaux GN-atlas ou ds GN-citizen sont publiés, n'hésitez à compléter cette liste pour les partager avec la communauté : https://lite.framacalc.org/9efn-geonature-users |
|
Merci @camillemonchicourt pour ton retour et d'avoir rajouté ces portails . Oui effectivement il faut qu'on rajoute les nouveaux GN-atlas/GN-citizen à ce document . On va refaire une passe |
|
Cette PR ne fonctionnait pas dans différents contextes donc @amandine-sahl a du la reprendre. Voir #475 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dans le cadre d'un travail réalisé avec la LPO PACA , ont nous a remonté des latences de chargement au niveau de la page d'accueil. Avec @mvergez on a ciblé une requête qui prenait particulièrement du temps.
Ainsi, la requête dans cette fonction (lien ici) dure actuellement environ 7sec
La nouvelle requête proposée dans cette PR permet de passer à 0.34 sec.
Le contexte d'analyse s'est fait dans DBeaver pour les bases de données de :
Reviewed-by: andriac et @mvergez (Natural Solutions)