Revision dd757ffde0117af50ea22c295469389f84501f11 authored by Maysam Yabandeh on 20 August 2019, 18:38:15 UTC, committed by myabandeh on 18 September 2019, 21:39:45 UTC
Summary:
The comments of snap_refresh_nanos advertise that the snapshot refresh feature will be disabled when the option is set to 0. This contract is however not honored in the code: https://github.com/facebook/rocksdb/pull/5278
The patch fixes that and also adds an assert to ensure that the feature is not used when the option  is zero.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5724

Differential Revision: D16918185

Pulled By: maysamyabandeh

fbshipit-source-id: fec167287df7d85093e087fc39c0eb243e3bbd7e
1 parent fdc56e2
History
File Mode Size
cf_options.cc -rw-r--r-- 10.4 KB
cf_options.h -rw-r--r-- 8.9 KB
db_options.cc -rw-r--r-- 14.6 KB
db_options.h -rw-r--r-- 3.2 KB
options.cc -rw-r--r-- 25.4 KB
options_helper.cc -rw-r--r-- 90.3 KB
options_helper.h -rw-r--r-- 9.0 KB
options_parser.cc -rw-r--r-- 29.9 KB
options_parser.h -rw-r--r-- 5.1 KB
options_sanity_check.cc -rw-r--r-- 1.1 KB
options_sanity_check.h -rw-r--r-- 1.7 KB
options_settable_test.cc -rw-r--r-- 21.3 KB
options_test.cc -rw-r--r-- 77.9 KB

back to top