diff options
| author | Jay Zhuang <zjay@fb.com> | 2022-05-19 11:04:21 -0700 |
|---|---|---|
| committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | 2022-05-19 11:04:21 -0700 |
| commit | c6d326d3d7912c963b16dce4a7e5e5744da470ab (patch) | |
| tree | eaa928e2b86b55b965413b91a9c7a6a9a7e0894e /tools | |
| parent | dde774db646d74054a312878244206431ee939e4 (diff) | |
Track SST unique id in MANIFEST and verify (#9990)
Summary:
Start tracking SST unique id in MANIFEST, which is used to verify with
SST properties to make sure the SST file is not overwritten or
misplaced. A DB option `try_verify_sst_unique_id` is introduced to
enable/disable the verification, if enabled, it opens all SST files
during DB-open to read the unique_id from table properties (default is
false), so it's recommended to use it with `max_open_files = -1` to
pre-open the files.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/9990
Test Plan: unittests, format-compatible test, mini-crash
Reviewed By: anand1976
Differential Revision: D36381863
Pulled By: jay-zhuang
fbshipit-source-id: 89ea2eb6b35ed3e80ead9c724eb096083eaba63f
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/db_crashtest.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index 9c013734f..756510109 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -173,6 +173,7 @@ default_params = { "adaptive_readahead": lambda: random.choice([0, 1]), "async_io": lambda: random.choice([0, 1]), "wal_compression": lambda: random.choice(["none", "zstd"]), + "verify_sst_unique_id_in_manifest": 1, # always do unique_id verification } _TEST_DIR_ENV_VAR = 'TEST_TMPDIR' |