Skip to content

Commit

Permalink
Revert an uncessary status code check skipping
Browse files Browse the repository at this point in the history
Summary:
facebook#7452 added an uncessary skip for status code checking. Revert it.

Test Plan: Watch CI to finish
  • Loading branch information
siying committed Sep 29, 2020
1 parent d08a900 commit cc75c33
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions db/version_set.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1836,8 +1836,6 @@ void Version::Get(const ReadOptions& read_options, const LookupKey& k,
GetPerfLevel() >= PerfLevel::kEnableTimeExceptForMutex &&
get_perf_context()->per_level_perf_context_enabled;
StopWatchNano timer(env_, timer_enabled /* auto_start */);
// Something feels not right here. Should investigate more.
status->PermitUncheckedError();
*status = table_cache_->Get(
read_options, *internal_comparator(), *f->file_metadata, ikey,
&get_context, mutable_cf_options_.prefix_extractor.get(),
Expand Down

0 comments on commit cc75c33

Please sign in to comment.