Fix argument mismatch in BlockBasedTableBuilder (#3974)

Summary:
The sixth argument should be `key_includes_seq` bool, the seventh a `GetContext*`. We were mistakenly passing the `GetContext*` as the sixth argument and relying on the default (nullptr) for the seventh. This would make statistics inaccurate, at least.

Blame: 402b7aa0
Closes https://github.com/facebook/rocksdb/pull/3974

Differential Revision: D8344907

Pulled By: ajkr

fbshipit-source-id: 3ad865a0541d6d30f75dfc726352788118cfe12e
main
Andrew Kryczka 7 years ago committed by Facebook Github Bot
parent 9c7da963bc
commit 9d347332fb
  1. 2
      table/block_based_table_reader.cc

@ -2209,7 +2209,7 @@ Status BlockBasedTable::Get(const ReadOptions& read_options, const Slice& key,
} else { } else {
BlockIter biter; BlockIter biter;
NewDataBlockIterator(rep_, read_options, iiter->value(), &biter, false, NewDataBlockIterator(rep_, read_options, iiter->value(), &biter, false,
get_context); true /* key_includes_seq */, get_context);
if (read_options.read_tier == kBlockCacheTier && if (read_options.read_tier == kBlockCacheTier &&
biter.status().IsIncomplete()) { biter.status().IsIncomplete()) {

Loading…
Cancel
Save