yugabyte/yugabyte-db

Audit use of ForceRocksDBCompactInTest and rename

Open

#5.611 aberto em 8 de set. de 2020

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C (1.003 forks)batch import
area/docdbgood first issuekind/enhancementpriority/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-1508 Despite this methods name, we seem to have used this method in void TabletServiceAdminImpl::FlushTablets

void Tablet::ForceRocksDBCompactInTest() {
  if (regular_db_) {
    docdb::ForceRocksDBCompact(regular_db_.get());
  }
  if (intents_db_) {
    CHECK_OK(intents_db_->Flush(rocksdb::FlushOptions()));
    docdb::ForceRocksDBCompact(intents_db_.get());
  }
}

One obvious issue with this method in production code is the use of CHECK_OK. This only affects admin CLI flush commands, so this isn't a huge deal, but we should fix this regardless and ensure there are no other obvious or subtle issues with using this in production

cc @spolitov

Guia do colaborador

Direção de pesquisa
Audite todos os usos de ForceRocksDBCompactInTest na base de código, particularmente em TabletServiceAdminImpl::FlushTablets. Identifique quaisquer problemas com CHECK OK e garanta que o método seja renomeado apropriadamente. Consulte a issue e o Jira DB 1508 vinculado para discussão.
Pilha de tecnologia
cpp
Domain
databasebackend
Tipo Issue
Refatoração
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.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55