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

Latence lors du chargement des articles #818

Closed
RochDLY opened this issue Feb 6, 2023 · 6 comments
Closed

Latence lors du chargement des articles #818

RochDLY opened this issue Feb 6, 2023 · 6 comments

Comments

@RochDLY
Copy link
Collaborator

RochDLY commented Feb 6, 2023

Comportement remarqué par @marviro : Lorsqu'un compte a beaucoup d'articles, la page [Articles] de Stylo met plusieurs dizaines de secondes avant d'afficher les résultats de la requête.
Est-il possible d'optimiser la requête pour un affichage plus rapide ?

@RochDLY RochDLY added this to the v2.1 — Workspaces milestone Feb 6, 2023
@marviro
Copy link
Member

marviro commented Feb 6, 2023

Pour être plus précis: 120 articles, 18 secondes d'attente pour charger la page avec la liste des articles ;)

Avec @thom4parisot on s'est dit qu'il serait possible de limiter les requêtes. Deux pistes:

  1. paginer (prendre les dernier 20 articles par ex et avoir un bouton pour charger les autres, ou alors charger lors du scroll). Le problème de cette option est ce qui se passe au moment de la recherche par titre...
  2. Chercher pour chaque article seulement les informations nécessaires (titre? date? à définir) et charger les infos supplémentaire seulement lorsqu'on clique dessus (typiquement les versions)

@ggrossetie
Copy link
Collaborator

Pour être plus précis: 120 articles, 18 secondes d'attente pour charger la page avec la liste des articles ;)

Cela me semble vraiment lent pour 120 articles. On fait combien de requêtes ? Est-ce que ce n'est pas à cause des partages de compte que c'est lent ?

@marviro
Copy link
Member

marviro commented Feb 8, 2023

Il est possible que mon ordinateur soit un peu coupable. Mais avec un ordi plus récent 8 secondes... donc tjs trop.

Je crois qu'il y a des requêtes sur toutes les versions de tous les articles...

Les partages de comptes : je sais pas, mais je crois que non, car les requêtes sont faite juste au moment où on clique... Au moins c'est ce que j'imagine

@antoinentl
Copy link
Member

antoinentl commented Mar 2, 2023

Je rajoute mon retour :

  • stylo-dev : 14 secondes pour 173 articles, avec un ordinateur relativement récent et puissant
  • stylo-prod : 4 secondes pour 172 articles, même situation

@ggrossetie
Copy link
Collaborator

A faire :

  • Mesurer précisément le temps de chargement (i.e., où est ce qu'on passe le plus de temps ?)
  • Trouver des pistes d'optimisations

Questions ouvertes :

  • Est-ce qu'il serait envisageable de ne charger que les X derniers articles (par date de modification) et de charger le reste des articles de manière asynchrone ?
  • Est-ce qu'il serait envisageable d'afficher uniquement X articles avec un button permettant d'en charger plus (ou une pagination) ?

@ggrossetie
Copy link
Collaborator

Je pense que les performance sont maintenant acceptable. Il faudrait refaire des tests sur les comptes ayant beaucoup d'articles afin de mesurer le gain et voir comment on peut continuer d'améliorer le temps de chargement.

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

No branches or pull requests

4 participants