Suivi — Étiquettes Étiquettes populaires et performances

#1965 Posté par  (site web personnel) . État de l’entrée : corrigée. Assigné à Benoît Sibaud. Licence CC By‑SA.
Étiquettes : aucune
1
16
jan.
2021

Une campagne récente de ré-étiquetage massif a montré un effet des étiquettes populaires dans le bas de page :

https://pix.toile-libre.org/upload/original/1610822726.png

(l'évolution de la courbe correspond en fait à l'évolution temps de rendu de _layouts/_footer.html.haml )

Bref on va avoir besoin d'une mise en cache (pour éviter la sensibilité sur les étiquetages du dernier mois).

  • # Double jointure de la mort

    Posté par  (site web personnel) . Évalué à 3 (+0/-0).

    Le souci vient d'une double jointure, que j'ai involontairement introduite précédemment pour masquer les étiquettes masquées et les étiquettes des contenus masqués.

    commit 340eacd82cf82b9bffc089ad5c02383dadd76a71
Author: Benoît Sibaud <oumph@linuxfr.org>
Date:   Wed Nov 21 20:47:14 2018 +0100

    Popular tags based only on public nodes

diff --git a/app/models/tag.rb b/app/models/tag.rb
index dc97553d..39f31643 100644
--- a/app/models/tag.rb
+++ b/app/models/tag.rb
@@ -20,8 +20,10 @@ class Tag < ActiveRecord::Base
   scope :footer, -> {
     select([:name]).joins(:taggings).
                     where(public: true).
-                    where("created_at > ?", 1.month.ago).
-                    group(:tag_id).
+                    joins(:nodes).
+                    where("nodes.public = 1").
+                    where("taggings.created_at > ?", 1.month.ago).
+                    group("taggings.tag_id").
                     order(Arel.sql("COUNT(*) DESC")).
                     limit(12)
   }

    (:nodes contient déjà une jointure via taggings, donc on fait une double jointure via taggings, et dès qu'il y a beaucoup d'étiquetages, le résultat est monstrueux… exemple: "43454465 rows in set (1 min 41.74 sec)" avec un essai d'insertion de 6500 étiquetages…)

    Une correction serait :

       scope :footer, -> {
-    select([:name]).joins(:taggings).
-                    where(public: true).
-                    joins(:nodes).
-                    where("nodes.public = 1").
+    select([:name]).visible.
+                    joins(:nodes).merge(Node.visible).
                     where("taggings.created_at > ?", 1.month.ago).
                     group("taggings.tag_id").
                     order(Arel.sql("COUNT(*) DESC")).

Envoyer un commentaire

Suivre le flux des commentaires

Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.