Skip to content

Commit

Permalink
Remove the orphan assert on !need_log_sync
Browse files Browse the repository at this point in the history
Summary:
We initially had disabled support for write_options.sync when concurrent_prepare_ is set. We later added this support but the statement that asserts this combination is not used was left there. This patch cleans it up.
Closes facebook#2642

Differential Revision: D5496101

Pulled By: maysamyabandeh

fbshipit-source-id: becbc503446f2a51bee24cc861958c090c724ec2
  • Loading branch information
Maysam Yabandeh authored and facebook-github-bot committed Jul 26, 2017
1 parent fe1a555 commit 30b58cf
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion db/db_impl_write.cc
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,6 @@ Status DBImpl::WriteImpl(const WriteOptions& write_options,
need_log_dir_sync, last_sequence + 1);
}
} else {
assert(!need_log_sync && !need_log_dir_sync);
if (status.ok() && !write_options.disableWAL) {
PERF_TIMER_GUARD(write_wal_time);
// LastToBeWrittenSequence is increased inside WriteToWAL under
Expand Down

0 comments on commit 30b58cf

Please sign in to comment.