tarantool/tarantool

`O_DSYNC` should be used instead of `O_SYNC` for WAL files when available

Open

#10.784 geöffnet am 5. Nov. 2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Lua (406 Forks)batch import
buggood first issue

Repository-Metriken

Stars
 (3.633 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 13T 2h) (82 gemergte PRs in 30 T)

Beschreibung

Currently, we always use O_SYNC for ensuring for write integrity: https://github.com/tarantool/tarantool/blob/8806b7b561d58d1cef79e6c9c723827f4ce67ed8/src/box/wal.c#L441-L442

However, we only need data integrity guarantees, so O_DSYNC is sufficient, when it is available. We even have a legacy macro defined for this: https://github.com/tarantool/tarantool/blob/8806b7b561d58d1cef79e6c9c723827f4ce67ed8/src/trivia/config.h.cmake#L107-L115

It seems like there was a refactoring during which this logic was lost.

Contributor Guide

Research-Richtung
Die Änderung beinhaltet das Ersetzen der Verwendung von O SYNC durch das Legacy Makro TARANTOOL O DSYNC oder Ähnliches in src/box/wal.c. Beziehen Sie sich auf die vorhandene Makrodefinition in src/trivia/config.h.cmake, um die korrekte Verwendung sicherzustellen. Stellen Sie sicher, dass die Änderung kompiliert und relevante Tests besteht, insbesondere solche, die die Integrität der WAL Datei betreffen.
Tech Stack
c
Domain
databasebackend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70