From 16d103d35b2eca58bb5c5750d533f1e7078fbdd3 Mon Sep 17 00:00:00 2001 From: Huisheng Liu Date: Tue, 10 Nov 2020 11:13:10 -0800 Subject: [PATCH] fix read_amp_bytes_per_bit field size (#7651) Summary: The field in BlockBasedTableOptions is 4 bytes: // Default: 0 (disabled) uint32_t read_amp_bytes_per_bit = 0; Pull Request resolved: https://github.com/facebook/rocksdb/pull/7651 Reviewed By: ltamasi Differential Revision: D24844994 Pulled By: riversand963 fbshipit-source-id: e2695e55532256ef8996dd6939cad06987a80293 --- table/block_based/block_based_table_factory.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/table/block_based/block_based_table_factory.cc b/table/block_based/block_based_table_factory.cc index 9e789a6ec..ca6a27f40 100644 --- a/table/block_based/block_based_table_factory.cc +++ b/table/block_based/block_based_table_factory.cc @@ -359,7 +359,7 @@ static std::unordered_map OptionTypeFlags::kNone}}, {"read_amp_bytes_per_bit", {offsetof(struct BlockBasedTableOptions, read_amp_bytes_per_bit), - OptionType::kSizeT, OptionVerificationType::kNormal, + OptionType::kUInt32T, OptionVerificationType::kNormal, OptionTypeFlags::kNone}}, {"enable_index_compression", {offsetof(struct BlockBasedTableOptions, enable_index_compression),