yugabyte/yugabyte-db

[DocDB] Add flag for configuring rocksdb bytes_per_sync option

Open

#10.822 aperta il 9 dic 2021

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)C (1003 fork)batch import
area/docdbgood first issuekind/enhancementpriority/medium

Metriche repository

Star
 (8229 star)
Metriche merge PR
 (Merge medio 17g 21h) (92 PR mergiate in 30 g)

Descrizione

Jira Link: DB-590

Description

  // Allows OS to incrementally sync files to disk while they are being
  // written, in the background. Issue one request for every bytes_per_sync
  // written. 0 turns it off.
  // Default: 0
  uint64_t bytes_per_sync = 0;

DBOptions SanitizeOptions(const std::string& dbname, const DBOptions& src) {
...
  if (result.rate_limiter.get() != nullptr) {
    if (result.bytes_per_sync == 0) {
      result.bytes_per_sync = 1024 * 1024;
    }
  }
...

Currently, we don't set bytes_per_sync explicitly, but we use a rate limiter, so it leads to sync every 1MB (independently for each file) at rocksdb level.

Guida contributor

Direzione di ricerca
Esamina la logica di sanitizzazione delle DBOptions e comprendi come vengono esposti i flag esistenti. Quindi, aggiungi un nuovo gflag per bytes per sync e integralo nella funzione di sanitizzazione.
Tech stack
cpp
Dominio
databasebackend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++rocksdbGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60