yugabyte/yugabyte-db

[DocDB] Add flag for configuring rocksdb bytes_per_sync option

Open

#10.822 geöffnet am 9. Dez. 2021

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C (1.003 Forks)batch import
area/docdbgood first issuekind/enhancementpriority/medium

Repository-Metriken

Stars
 (8.229 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 17T 21h) (92 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die DBOptions Bereinigungslogik und verstehen Sie, wie vorhandene Flags exponiert werden. Fügen Sie dann ein neues gflag für bytes per sync hinzu und integrieren Sie es in die Bereinigungsfunktion.
Tech Stack
cpp
Domain
databasebackend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++rocksdbGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60