Skip to content

Commit ebaf09d

Browse files
hliu18riversand963
authored andcommitted
fix read_amp_bytes_per_bit field size (facebook#7651)
Summary: The field in BlockBasedTableOptions is 4 bytes: // Default: 0 (disabled) uint32_t read_amp_bytes_per_bit = 0; Pull Request resolved: facebook#7651 Reviewed By: ltamasi Differential Revision: D24844994 Pulled By: riversand963 fbshipit-source-id: e2695e55532256ef8996dd6939cad06987a80293
1 parent b6db05d commit ebaf09d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

table/block_based/block_based_table_factory.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,7 @@ static std::unordered_map<std::string, OptionTypeInfo>
333333
OptionTypeFlags::kNone}},
334334
{"read_amp_bytes_per_bit",
335335
{offsetof(struct BlockBasedTableOptions, read_amp_bytes_per_bit),
336-
OptionType::kSizeT, OptionVerificationType::kNormal,
336+
OptionType::kUInt32T, OptionVerificationType::kNormal,
337337
OptionTypeFlags::kNone}},
338338
{"enable_index_compression",
339339
{offsetof(struct BlockBasedTableOptions, enable_index_compression),

0 commit comments

Comments
 (0)