Skip to content

Commit

Permalink
Fix rocksdb include definitions
Browse files Browse the repository at this point in the history
Summary:
composite_env_wrapper.h and fault_injection_fs.h are not
public RocksDB APIs. They should not start with "rocksdb/" include
definitions. This diff fixes it.

Reviewed By: pdillinger

Differential Revision: D41090954

fbshipit-source-id: 1fd5cd3
  • Loading branch information
Yoshinori Matsunobu authored and inikep committed May 18, 2023
1 parent 22106ef commit 212d743
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions storage/rocksdb/ha_rocksdb.cc
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@
#include "rocksdb/compaction_filter.h"
#include "rocksdb/compaction_job_stats.h"
#include "rocksdb/env.h"
#include "rocksdb/env/composite_env_wrapper.h"
#include "env/composite_env_wrapper.h"
#include "rocksdb/memory_allocator.h"
#include "rocksdb/persistent_cache.h"
#include "rocksdb/rate_limiter.h"
Expand All @@ -72,7 +72,7 @@
#include "rocksdb/trace_reader_writer.h"
#include "rocksdb/utilities/checkpoint.h"
#include "rocksdb/utilities/convenience.h"
#include "rocksdb/utilities/fault_injection_fs.h"
#include "utilities/fault_injection_fs.h"
#include "rocksdb/utilities/memory_util.h"
#include "rocksdb/utilities/sim_cache.h"
#include "rocksdb/utilities/write_batch_with_index.h"
Expand Down

0 comments on commit 212d743

Please sign in to comment.