`O_DSYNC` should be used instead of `O_SYNC` for WAL files when available · tarantool/tarantool#10784 | Good First Issue

(0 留言) (0 反應) (0 負責人)Lua (406 fork)batch import

buggood first issue

倉庫指標

Star: (3,633 star)
PR 合併指標: (平均合併 13天 2小時) (30 天內合併 82 個 PR)

描述

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.

貢獻者指南

研究方向: 本次變更涉及在 src/box/wal.c 中將 O SYNC 的使用替換為遺留宏 TARANTOOL O DSYNC 或類似宏。請參考 src/trivia/config.h.cmake 中現有的宏定義以確保正確使用。確保變更能夠編譯並通過相關測試，特別是與 WAL 檔案完整性相關的測試。
技術棧: c
領域: databasebackend
議題類型: 錯誤
難度: 1
預計時間: 1 小時以內
活動狀態: 新近可參與
清晰度: 清晰
前置要求: C
新手友善度: 70