Skip to content

Commit

Permalink
Fix TSAN timeout in MergeOperatorPinningTest.Randomized/x test
Browse files Browse the repository at this point in the history
Summary:
[FB - Internal]
MergeOperatorPinningTest.Randomized/x tests are frequently failing with timeouts when run with tsan, as they are exceeding 10 minute limit for tests. The tests are in turn getting disabled due to frequent failures.
I halved the number of rounds to make the test complete sooner. This reduces the number of testing iterations a little, but it still is much better than totally letting the test be disabled.
Closes facebook#3523

Differential Revision: D7031498

Pulled By: sagar0

fbshipit-source-id: 9a694f2176b235259920a42bf24bca5346f7cff1
  • Loading branch information
sagar0 authored and facebook-github-bot committed Mar 3, 2018
1 parent db2445a commit 15f55e5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion db/db_merge_operator_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -284,7 +284,7 @@ TEST_P(MergeOperatorPinningTest, Randomized) {
Random rnd(301);
std::map<std::string, std::string> true_data;

const int kTotalMerges = 10000;
const int kTotalMerges = 5000;
// Every key gets ~10 operands
const int kKeyRange = kTotalMerges / 10;
const int kOperandSize = 20;
Expand Down

0 comments on commit 15f55e5

Please sign in to comment.