yugabyte/yugabyte-db

Audit use of ForceRocksDBCompactInTest and rename

Open

#5 611 ouverte le 8 sept. 2020

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C (1 003 forks)batch import
area/docdbgood first issuekind/enhancementpriority/medium

Métriques du dépôt

Stars
 (8 229 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (92 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Auditez toutes les utilisations de ForceRocksDBCompactInTest dans la base de code, en particulier dans TabletServiceAdminImpl::FlushTablets. Identifiez tout problème avec CHECK OK et assurez vous que la méthode est renommée de manière appropriée. Reportez vous à l'issue et au Jira DB 1508 lié pour la discussion.
Stack technique
cpp
Domaine
databasebackend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55