yugabyte/yugabyte-db

[DocDB] AsyncClientInitializer does not set promise on early shutdown

Open

#21.316 aberto em 5 de mar. de 2024

Ver no GitHub
 (7 comments) (0 reactions) (1 assignee)C (1.003 forks)batch import
area/docdbgood first issuekind/bugpriority/medium

Métricas do repositório

Stars
 (8.229 stars)
Métricas de merge de PR
 (Mesclagem média 17d 21h) (92 fundiu PRs em 30d)

Description

Jira Link: DB-10220

Description

if we try to gracefully shutdown before the client is Initialized then we will hang waiting on Future::Get. This is because async client initializer does not set the promise if we shutdown before the client is ready. Instead we should return Result<YbClient*> and set bad status on shutdown. Not a production issue since we don't shutdown gracefully anyway.

Issue Type

kind/bug

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.

Guia do colaborador

Direção de pesquisa
Examine a implementação do AsyncClientInitializer para entender por que a promise não é definida durante o desligamento antecipado. Considere modificar o método para retornar Result<YbClient*> e definir um status de erro no desligamento, em vez de travar.
Pilha de tecnologia
c
Domain
backenddatabasedistributed systems
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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.
Principalmente claro
Prerequisites
C++async programming
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60