Draft: Ajout d'un cache
Liste des choses à vérifier
Merci de vérifier is votre Merge Request (MR) satisfait bien aux conditions suivantes (mettre un 'x' pour cocher la case) :
-
Le code et les commentaires sont en anglais -
Nous avons ajouté nos noms dans le fichier AUTHORS.md
-
Nous avons ajouté quelques mots d'anglais dans le CHANGELOG.md
pour résumer notre travail -
Si nous avons utilisé de nouveaux modules python, nous les avons ajouté dans le fichier requirements.txt
-
Nous avons effectué quelques tests (et les avons éventuellement ajoutés au dépôt) -
Nous avons ajouté, modifié et revu la documentation, si nécessaire. On peut effectuer une MR sur la page wiki de FlOpEDT, comme expliqué ici : https://stackoverflow.com/a/38537453 -
Nous n'avons pas ajouté d'affichage dans les consoles (ni dans celle de django (avec print
), ni dans celle du navigateur (avec desconsole.log
))
Nouvelles dépendances
Des dépendances nouvelles ont-elles été ajoutées ?
-
Oui -
Non
Type de Merge Request
Merci d'indiquer le type de votre MR (mettre un 'x' pour cocher la case) :
-
Réparation de bug -
Fonctionnalité -
Mise à jour de style sur le code (formatage, renommage) -
Refactoring (pas de changement fonctionnel, ni de changement d'API) -
Autre (à décrire) :
Quel est le comportement actuel ?
Actuellement, lors de la selection d'une semaine, des requetes api sont effectué est sont plutot longue (entre 5s et 10s)
Quel est le nouveau comportement ?
- Mise en cache pendant 20 min des requetes nécessaire au changement d'une semaine.
Est-ce que cette MR casse une autre fonctionnalité ?
-
Oui -
Non
Est-ce que cette MR contient de nouvelles migrations de base ?
-
Oui -
Non
Informations supplémentaires ?
Pour que la fonctionnalité de cache fonctionne, il faut que "redis" tourne sur la machine qui héberge FlopEDT. https://redis.io/docs/getting-started/installation/install-redis-on-linux/
ou
docker run -p "6379:6379" -d redis:7.0.11
Edited by ChristopherBiamonte