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

Performances affichages des bilans(ventes) (encore) très long #121

Closed
darnuria opened this issue Mar 6, 2017 · 3 comments
Closed

Performances affichages des bilans(ventes) (encore) très long #121

darnuria opened this issue Mar 6, 2017 · 3 comments

Comments

@darnuria
Copy link
Contributor

darnuria commented Mar 6, 2017

Sur de longues périodes les bilans des ventes sont très long à afficher plus de 7500ms sur une machine virtuelle avec 4go ram, 4processeurs en réseau local pour 2ans d'historique de la Petite Rockette.

Pistes de réflexion pour améliorer cela:

  • Préparer dans une requête SQL l'affichage par type d'objets ou chaque ligne représente un type d'objet (+ requêtes parallélisables par le gestionnaire de base de données, - requêtes complexe, + simple coté client - comme actuellement UI figée le temps que ça calcule)
  • Déléguer au client les opérations sur les ensembles d'objets et faire des requêtes plus génériques. (+ asynchrone + requêtes en parallèle, + simple coté serveur, - complexe coté client)
@darnuria darnuria changed the title Performances affichages des bilans(ventes) très long Performances affichages des bilans(ventes) (encore) très long Mar 6, 2017
@darnuria darnuria modified the milestone: Version 0.2.0 Mar 7, 2017
@darnuria
Copy link
Contributor Author

darnuria commented Mar 18, 2017

Youhou je bosse sur la prochaine PR on tombe à 6643msec pour afficher depuis 18/03/2013 au 18/03/2017!

Tandis que sur master on met 11456msec pour la même période.

On sais qui va être heureux. ;p

@darnuria
Copy link
Contributor Author

darnuria commented Mar 18, 2017

Je propage les modifications pour la sélection point de vente par point de vente et je pousse une PR.

Au passage ça nettoie bien le code du bilan.

@darnuria
Copy link
Contributor Author

darnuria commented Mar 18, 2017

Enfin résolus dans #145
On passe à 3565msec pour afficher l'historique complet de la rockette ça me parait enfin bien!

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

1 participant