tarantool/tarantool

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

Open

#10 784 ouverte le 5 nov. 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Lua (406 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (3 633 stars)
Métriques de merge PR
 (Merge moyen 13j 2h) (82 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Le changement implique de remplacer l'utilisation de O SYNC par la macro héritée TARANTOOL O DSYNC ou similaire dans src/box/wal.c. Reportez vous à la définition de macro existante dans src/trivia/config.h.cmake pour garantir une utilisation correcte. Assurez vous que le changement compile et passe les tests pertinents, en particulier ceux liés à l'intégrité des fichiers WAL.
Stack technique
c
Domaine
databasebackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70