Skip to content

Unlock Aggregate : cache global #4629

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

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AntoineAugusti
Copy link
Member

Fixes #4621

Met en cache de manière globale les réponses des feeds d'un flux aggregate dans le proxy.

En testant cette branche sur prochainement avec hey j'obtiens des 503 et je provoque un redémarrage… Je ne sais pas vraiment pourquoi 🤔

@AntoineAugusti AntoineAugusti requested a review from a team as a code owner June 10, 2025 09:42
@thbar
Copy link
Contributor

thbar commented Jun 12, 2025

Merci !

En testant cette branche sur prochainement avec hey j'obtiens des 503 et je provoque un redémarrage… Je ne sais pas vraiment pourquoi 🤔

Ok j'irai tester, merci pour l'alerte !

@thbar
Copy link
Contributor

thbar commented Jun 12, 2025

@AntoineAugusti est-ce que c'est systématique sur tes tests le redémarrage ? As-tu déjà regardé d'un peu plus près à tout hasard ? (je ne vais pas regarder tout de suite). Merci !

@AntoineAugusti
Copy link
Member Author

AntoineAugusti commented Jun 12, 2025

@thbar Oui, c'était systématique : avec le cache chaud ou froid :/ Je n'ai pas regardé de plus près.

$ hey https://proxy.prochainement.transport.data.gouv.fr/resource/consolidation-nationale-irve-dynamique

@thbar
Copy link
Contributor

thbar commented Jun 12, 2025

@thbar Oui, c'était systématique : avec le cache chaud ou froid :/ Je n'ai pas regardé de plus près.

$ hey https://proxy.prochainement.transport.data.gouv.fr/resource/consolidation-nationale-irve-dynamique

Ok merci, j'irai regarder !

@thbar thbar added the IRVE Sujets liés aux données d'infrastructures de recharge pour véhicule électrique label Jun 12, 2025
@AntoineAugusti
Copy link
Member Author

En passant la machine de S à L, ça passe.

$ hey https://proxy.prochainement.transport.data.gouv.fr/resource/consolidation-nationale-irve-dynamique

Summary:
  Total:	22.0468 secs
  Slowest:	14.8571 secs
  Fastest:	0.5026 secs
  Average:	4.9570 secs
  Requests/sec:	9.0716
  

Response time histogram:
  0.503 [1]	|■
  1.938 [49]	|■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  3.373 [34]	|■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  4.809 [26]	|■■■■■■■■■■■■■■■■■■■■■
  6.244 [27]	|■■■■■■■■■■■■■■■■■■■■■■
  7.680 [18]	|■■■■■■■■■■■■■■■
  9.115 [13]	|■■■■■■■■■■■
  10.551 [10]	|■■■■■■■■
  11.986 [12]	|■■■■■■■■■■
  13.422 [8]	|■■■■■■■
  14.857 [2]	|■■


Latency distribution:
  10% in 1.0446 secs
  25% in 1.9773 secs
  50% in 4.1983 secs
  75% in 6.8991 secs
  90% in 10.5990 secs
  95% in 12.1095 secs
  99% in 13.8694 secs

Details (average, fastest, slowest):
  DNS+dialup:	0.0726 secs, 0.5026 secs, 14.8571 secs
  DNS-lookup:	0.0146 secs, 0.0000 secs, 0.0611 secs
  req write:	0.0025 secs, 0.0000 secs, 0.1604 secs
  resp wait:	4.8224 secs, 0.4759 secs, 14.7335 secs
  resp read:	0.0587 secs, 0.0238 secs, 0.1686 secs

Ça prend du temps car la réponse est assez grosse j'imagine.

@thbar thbar added ops Gestion des serveurs et de la production proxy labels Jun 20, 2025
@thbar thbar mentioned this pull request Jun 20, 2025
6 tasks
@thbar
Copy link
Contributor

thbar commented Jun 24, 2025

Je vais repasser dessus cette semaine.

@thbar
Copy link
Contributor

thbar commented Jun 25, 2025

Je m'en occupe à mon retour de congés (vu avec @stephane-pignal @AurelienC maintenant qu'un a une beta version interne) ; il y a des points que je vais modifier.

@thbar thbar marked this pull request as draft June 25, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IRVE Sujets liés aux données d'infrastructures de recharge pour véhicule électrique ops Gestion des serveurs et de la production proxy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proxy IRVE dynamique à optimiser / mettre en cache
2 participants