zulip/zulip

Optimize topic cache calculation for rendering `near`/`with` links in automated notices after moving topics.

Ouverte

#33 276 ouverte le 31 janv. 2025

Voir sur GitHubGuide contributeur
 (24 commentaires) (0 réaction) (0 personne assignée)Python (7 339 forks)batch import
area: message-editing (topics)area: performancehelp wanted

Métriques du dépôt

Stars
 (19 672 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

This requires #30644 to be merged. See https://github.com/zulip/zulip/pull/30644#discussion_r1785332141 for details, but we might be able to use message IDs we just generated to prepopulate the topic cache to avoid extra queries when moving messages.

It might not be worth it; the queries are cheap and it would be potentially messy plumbing. I won't be certain until I see what the plumbing needs to look like.

But given that message-moving is a high importance code path that already has a lot of queries, it could be worth it.

Guide contributeur

Direction de recherche
Explorez l'implémentation dans #30644 et la discussion dans le commentaire de PR lié. Déterminez si l'utilisation des ID de message provenant des messages générés peut pré remplir le cache des sujets pour éviter des requêtes supplémentaires lors du déplacement des messages. Évaluez la complexité de la plomberie requise et les compromis. Les fichiers tels que le code de déplacement des messages et l'implémentation du cache des sujets dans le serveur Zulip doivent être examinés.
Stack technique
pythondjango
Domaine
backend
Type d'issue
Performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonDjangoGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30