8000 Feat: use cache for index page stats by jpm-cbna · Pull Request #398 · PnX-SI/GeoNature-atlas · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Feat: use cache for index page stats #398

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

Closed
wants to merge 1 commit into from
Closed

Conversation

jpm-cbna
Copy link
Contributor

See #269. Supersedes PR #270.

@TheoLechemia
Copy link
Member

Je me pose la question de l'utilisation de Flask-cache pour ce genre de besoin (https://flask.palletsprojects.com/en/2.0.x/patterns/caching/).
ça nécessiterait de passer ces calculs de stats en API (ce qui accélérerait largement le chargement initial).
Il y a deux cas ou le cache est utilisé :

  • les stats global
  • les stats par rang taxonomiques (qui nécessite un accès à la conf applicative)

@jpm-cbna propose de garder un vm_stat pour le calcul des stats global et d'utiliser du cache pour les stats par rang.
Je me pose la question de hétérogénéité que cela introduit ?

@jpm-cbna
Copy link
Contributor Author

@TheoLechemia propose une PR #400 utilisant Flask-Cache à la place de la VM vm_stat et de la table t_cache. Le fonctionnement de Flask-Cache simplifiant la base de données cela pourrait être une excellente alternative. À tester avec la base SINP PACA.

@amandine-sahl
Copy link
Contributor

A fermer au vu de #400 ?

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

Successfully merging this pull request may close these issues.

3 participants
0