Skip to content

Commit

Permalink
Exclude MergeInProgress status from errors in stress tests (facebook#…
Browse files Browse the repository at this point in the history
…6257)

Summary:
When called on transactions, MultiGet could return a legit MergeInProgress status. The patch excludes this case from errors.
Pull Request resolved: facebook#6257

Differential Revision: D19275787

Pulled By: maysamyabandeh

fbshipit-source-id: f7158229422af015947e592ae066b4273c9fb9a4
  • Loading branch information
Maysam Yabandeh authored and facebook-github-bot committed Jan 3, 2020
1 parent 7c98d71 commit 83957dc
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions db_stress_tool/no_batched_ops_stress.cc
Original file line number Diff line number Diff line change
Expand Up @@ -242,6 +242,9 @@ class NonBatchedOpsStressTest : public StressTest {
} else if (s.IsNotFound()) {
// not found case
thread->stats.AddGets(1, 0);
} else if (s.IsMergeInProgress() && use_txn) {
// With txn this is sometimes expected.
thread->stats.AddGets(1, 1);
} else {
// errors case
fprintf(stderr, "MultiGet error: %s\n", s.ToString().c_str());
Expand Down

0 comments on commit 83957dc

Please sign in to comment.