Question : qu'est-ce qui a été fait sur la v15.6 en terme de performances ?

[SpaceFox]

Je viens de voir que la page d'accueil affiche, pour un article :

par vous, Eskimon et Situphen

Or, il me semblait qu'on avait fait sauter la détection du "vous" pour des raisons de performances.

Est-ce qu'on est sûr que nos améliorations ont bien été mergées dans la version qui est en prod aujourd'hui ?

[Situphen]

Non, si je me souviens bien on l'avait fait sauté lors de la mise en place du cache (car sinon ça aurait été trop compliqué) mais du coup on a du le remettre lorsqu'on a enlevé le cache !

Franchement, ça ne doit pas affecter grand chose en terme de performances, c'est simplement un if sur deux variables !

[SpaceFox]

En fait ma question portait plus sur le fait qu'on ait pu annuler des
optimisations réelles suite à cet suppression du cache.

Le 8 juillet 2015 17:36, Situphen notifications@github.com a écrit :

Non, si je me souviens bien on l'avait fait sauté lors de la mise en place
du cache (car sinon ça aurait été trop compliqué) mais du coup on a du le
remettre lorsqu'on a enlevé le cache !

Franchement, ça ne doit pas affecter grand chose en terme de performances,
c'est simplement un if sur deux variables
https://github.com/zestedesavoir/zds-site/blob/dev/templates/article/includes/article_item.part.html#L15
!

—
Reply to this email directly or view it on GitHub
#2904 (comment)
.

[Situphen]

Franchement, ça ne doit pas affecter grand chose en terme de performances, c'est simplement un if sur deux variables !

[SpaceFox]

En général, pas sur ce point en particulier...

[Situphen]

A part peut-être la suppression de get_reaction_count, il n'y a pas eu d'améliorations autre que le cache, si ?

[SpaceFox]

Il n'y avait pas des tas d'améliorations sur les requêtes ?!

[Situphen]

Ça n'a pas été fait en même temps que le cache. Ça a été fait par #2785 !

[SpaceFox]

Mais c'est pas du tout ce que je dit !
Le post d'origine ne parle pas de cache et dit très explicitement :

Question : qu'est-ce qui a été fait sur la v15.6 en terme de performances ?

Je ne comprend même pas comment on peut en arriver à ce débat !

[Situphen]

Je crois qu'il n'y a que #2785 et #2724 concernant tout le site. Il y en a sûrement quelques autres liées à la recherche et l'indexation qui ont du améliorer les performances aussi.

[artragis]

globalement, nous avons abaissé pas mal le nombre de requêtes quand il y a des "j'aime" ou des badges à afficher. Mes précalculs ne concernent que ce que j'ai pu remarquer durant la zep12, pas autre chose. Notre bottleneck se situe au niveau des notifications qui chargent énormément de choses en lazy et sans trop regarder ce qu'ils font.

[pierre-24]

A priori réglé ;)

[artragis]

non

Le 22/07/2015 22:06, Pierre Beaujean a écrit :

A priori réglé ;)

—
Reply to this email directly or view it on GitHub
#2904 (comment).

[pierre-24]

ah :o

[Situphen]

Up ! Ça en est où ?

[Eskimon]

cette version date, ce ticket a encore raison d'être ?

[gustavi]

UP

[SpaceFox]

Plus de raison d'être ici.