Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18592 )
Change subject: [kudu-tool-test] more robust TestRebuildTserverByLocalReplicaCopy ...................................................................... [kudu-tool-test] more robust TestRebuildTserverByLocalReplicaCopy I recently noticed that kudu-tool-test has been failing often [1]. It turned out that in most cases that's a timeout in the TestRebuildTserverByLocalReplicaCopy scenario while waiting for 10000 rows to be written into the test table: Expected: (workload.rows_inserted()) >= (10000), actual: 8350 vs 10000 src/kudu/util/test_util.cc:376: Failure Failed Timed out waiting for assertion to pass. To address the flakiness, this patch adjusts the number of written rows for TSAN builds to be 1000. [1] http://dist-test.cloudera.org:8080/test_drilldown?test_name=kudu-tool-test Change-Id: Ic967f9d1eb732d2c735efc5b1d6edabbc275a3b7 Reviewed-on: http://gerrit.cloudera.org:8080/18592 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Yingchun Lai <[email protected]> --- M src/kudu/tools/kudu-tool-test.cc 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Alexey Serbin: Verified Mahesh Reddy: Looks good to me, but someone else must approve Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18592 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic967f9d1eb732d2c735efc5b1d6edabbc275a3b7 Gerrit-Change-Number: 18592 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
