search

eric-taix / volley34-mobile

Application mobile pour le Volley34
3 stars 0 forks source link

Requêtes lentes #298

Open eric-taix opened 2 years ago

eric-taix commented 2 years ago

J'ouvre ce nouveau ticket pour référencer les requêtes qui prennent beaucoup de temps. Screenshot_1644624788

Lo0ping commented 2 years ago

J'ai commencé quelques optimisations, j'espère que j'ai pas tout cassé 😜 je n'ai déployé que sur APITest. j'attends vos retours

Lo0ping commented 2 years ago

@eric-taix, as tu pu test avec apitest.volley34.fr ?

eric-taix commented 2 years ago

Non j'avais zappé et j'avais d'autres choses (hors dev) à faire. J'essaie de prendre un peu de temps dans la journée pour tester

eric-taix commented 2 years ago

1er écran (http://apitest.volley34.fr/v1/clubs/VCV/equipes) => ERROR coté serveur 💥 . Donc impossible d'aller plus loin.

Lo0ping commented 2 years ago

ça devrait aller mieux, désolé

eric-taix commented 2 years ago

Toujours pas => http://apitest.volley34.fr/v1/clubs/VCV/equipes

Lo0ping commented 2 years ago

Etrange, il doit y avoir un truc qui change au moment ou je publie car ce matin, la requête passait sur mon site de dev local

Lo0ping commented 2 years ago

@eric-taix , c'etait la BDD de test qui déconnait, ça fonctionne à nouveau image

eric-taix commented 2 years ago

Ok maintenant voilà le nouveau endpoint en erreur: http://apitest.volley34.fr/v1/resultats/2022-C024/EX/0

Peut être que l'idée est de rajouter, dans une prochaine version, une option cachée pour basculer sur l'API de test plutôt que celle de production. Cela nous permettra de tester (cette fois çi et peut être d'autres fois si plusieurs cycles d'optimisation sont nécessaires...).

Lo0ping commented 2 years ago

C'est corrigé... vraiment désolé, les optimisations ont nécessité beaucoup de modification et je n'ai pas eu le temps de tout tester ... je vais tenter de monter un projet de tests automatisés.

eric-taix commented 2 years ago

Pas de soucis. Pour le test automatisé c'est ce que je me disais et j'ai essayé très rapidement entre midi et 2 d'utiliser postman mais je n'ai pas encore compris comment faire des tests...

eric-taix commented 2 years ago

Ok je viens de tester et à priori l'ensemble de l'application fonctionne.

Ci-après la liste des requêtes lentes mise à jour (donc celles qui dépasse les 1000ms), donc finalement mise à part /v1/classements qui prend encore 7 secondes et qui à priori a même augmenté, sinon les performances commencent à être bien meilleure et cela se ressent grandement sur l'application.

En conclusion je dirais que c'est suffisant pour l'instant en terme d'optimisation 🏅

Screenshot_1646851221