git-server-git.apps.pok.os.sepia.ceph.com Git

]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/commit

author	mrambacher <mrambach@gmail.com>
	Mon, 13 Sep 2021 15:45:13 +0000 (08:45 -0700)
committer	Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com>
	Mon, 13 Sep 2021 15:46:19 +0000 (08:46 -0700)
commit	dafa584fd10918ec322053a54fcb7f0f7c0ead9f
tree	0d4c1763daac066f6a1620f29dd29e50423e7f41	tree \| snapshot
parent	2a2b3e03a512405ceee356e15f542744d434ff24	commit \| diff

Change the File System File Wrappers to std::unique_ptr (#8618)

Summary:
This allows the wrapper classes to own the wrapped object and eliminates confusion as to ownership. Previously, many classes implemented their own ownership solutions. Fixes https://github.com/facebook/rocksdb/issues/8606

Pull Request resolved: https://github.com/facebook/rocksdb/pull/8618

Reviewed By: pdillinger

Differential Revision: D30136064

Pulled By: mrambacher

fbshipit-source-id: d0bf471df8818dbb1770a86335fe98f761cca193

Unnamed repository; edit this file 'description' to name the repository.

RSS Atom

db/db_basic_test.cc		diff \| blob \| history
env/file_system_tracer.h		diff \| blob \| history
file/prefetch_test.cc		diff \| blob \| history
include/rocksdb/file_system.h		diff \| blob \| history
tools/simulated_hybrid_file_system.cc		diff \| blob \| history
tools/simulated_hybrid_file_system.h		diff \| blob \| history
utilities/fault_injection_fs.cc		diff \| blob \| history
utilities/fault_injection_fs.h		diff \| blob \| history