Skip to content

Commit

Permalink
Fix TSAN version of TestRedisService.TestHMGetTiming
Browse files Browse the repository at this point in the history
Summary:
Applying so many parallel commands timeouts in TSAN build.
Changed to sync by smaller portions.

Test Plan: ybd tsan --cxx-test redisserver_redisserver-test --gtest_filter TestRedisService.TestHMGetTiming -n 100

Reviewers: mikhail, amitanand, timur

Reviewed By: timur

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D4657
  • Loading branch information
spolitov committed Apr 24, 2018
1 parent bb204e4 commit be18db9
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/yb/yql/redis/redisserver/redisserver-test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2598,6 +2598,9 @@ TEST_F(TestRedisService, TestHMGetTiming) {
string sj = std::to_string(j);
DoRedisTestInt(__LINE__, {"HSET", "parent_" + si, "subkey_" + sj, "value_" + sj}, 1);
}
if (IsSanitizer() || ((i & 0x7) == 0)) {
SyncClient();
}
}

SyncClient();
Expand Down

0 comments on commit be18db9

Please sign in to comment.