From e003d22526e6b3a5715316e0d1f77a5c3b2878cd Mon Sep 17 00:00:00 2001 From: Chinmay Kamat Date: Wed, 14 Mar 2018 00:48:11 -0700 Subject: [PATCH] Fix FaultInjectionTestEnv to work with DirectIO Summary: Implemented PositionedAppend() and use_direct_io() for TestWritableFile. With these changes, FaultInjectionTestEnv can be used with DirectIO enabled. Closes https://github.com/facebook/rocksdb/pull/3586 Differential Revision: D7244305 Pulled By: yiwu-arbug fbshipit-source-id: f6b7aece53daa0f9977bc684164a0693693e514c --- util/fault_injection_test_env.h | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/util/fault_injection_test_env.h b/util/fault_injection_test_env.h index 5d0ae6344..1992ab52e 100644 --- a/util/fault_injection_test_env.h +++ b/util/fault_injection_test_env.h @@ -68,6 +68,13 @@ class TestWritableFile : public WritableFile { virtual Status Flush() override; virtual Status Sync() override; virtual bool IsSyncThreadSafe() const override { return true; } + virtual Status PositionedAppend(const Slice& data, + uint64_t offset) override { + return target_->PositionedAppend(data, offset); + } + virtual bool use_direct_io() const override { + return target_->use_direct_io(); + }; private: FileState state_; -- GitLab