zulip/zulip

Prevent races resulting in incorrect search index results for newly sent messages

Aberta

#33.267 aberto em 30 de jan. de 2025

Ver no GitHubGuia do colaborador
 (16 comentários) (0 reação) (0 responsável)Python (7.339 forks)batch import
area: searchbugdesign reviewhelp wantedpriority: high

Métricas do repositório

Stars
 (19.672 estrelas)
Métricas de merge de PR
 (Mesclagem média 36d 6h) (110 fundiu PRs em 30d)

Description

There's a race when sending messages while looking at a search view, that clients will try to ask the server whether the message matches search terms after hearing about it. If there's any backlog in fts_update_log, the server may return incorrect results.

From the chat.zulip.org thread:

Options include:

  • Checking if any of the messages are in fts_update_log, and if so, processing them.
  • Checking if any of the messages are in fts_update_log, and if so, doing a direct calculation for whether they match.
  • Something else.

This might be somewhat high priority, since attempts to live-update a search view are fairly likely to trigger this race.

Guia do colaborador

Direção de pesquisa
Investigue a condição de corrida em fts update log quando mensagens são enviadas enquanto uma visualização de pesquisa está ativa. Examine o código relevante para processar atualizações de pesquisa de texto completo e considere opções como verificar se as mensagens estão em fts update log e processá las antes de servir os resultados da pesquisa. A discussão no tópico do chat.zulip.org vinculado (https://chat.zulip.org/#narrow/channel/9 issues/topic/inconsistent.20response.20from.20messages in narrow backend/near/1973652) pode fornecer mais insights. Considere implementar uma das soluções propostas e testar com um mock para reproduzir a condição de corrida.
Pilha de tecnologia
pythondjango
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitPython
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60