Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11945 )

Change subject: [tests] fix flake in TestRandomHistoryGCWorkload
......................................................................

[tests] fix flake in TestRandomHistoryGCWorkload

This patch fixes a flake most prominent in TSAN builds for the
RandomizedTabletHistoryGcITest.TestRandomHistoryGCWorkload scenario
of the tablet_history_gc-itest suite.

Before: dist_test --stress_cpu_threads=16: 256 out of 256 failed
  http://dist-test.cloudera.org/job?job_id=aserbin.1542394875.56611

After:  dist_test --stress_cpu_threads=16:   0 out of 256 failed
  http://dist-test.cloudera.org/job?job_id=aserbin.1542397028.71131

In addition, this patch also contains a minor code clean up of the
tablet_history_gc-itest suite to take advantage of the 'using'
declarations.

Change-Id: I8d146d4b83c8d488d3c1766a53fe4a4d322b6590
Reviewed-on: http://gerrit.cloudera.org:8080/11945
Reviewed-by: Adar Dembo <a...@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/integration-tests/tablet_history_gc-itest.cc
1 file changed, 15 insertions(+), 13 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/11945
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8d146d4b83c8d488d3c1766a53fe4a4d322b6590
Gerrit-Change-Number: 11945
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to