Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7209
Change subject: [registration-test] fix flake in TestTabletReports ...................................................................... [registration-test] fix flake in TestTabletReports Prior to the fix, the failure rate attributed to this flake was about 0.3% in 1K run: http://dist-test.cloudera.org//job?job_id=aserbin.1497582474.17739 After the fix, no failures in 1K run: http://dist-test.cloudera.org//job?job_id=aserbin.1497629325.23427 Both runs were of DEBUG build run with -stress_cpu_threads=8 flag. The crux of the fix is to use AssertEvetually instead of relying on a hard-coded delay to capture non-zero system catalog's metric reading updated upon processing tablet reports from TS. In addition to that, this patch includes a clean-up of the test code. Change-Id: I906465ad220236538175c80972ae055193f9bb45 --- M src/kudu/integration-tests/registration-test.cc 1 file changed, 57 insertions(+), 50 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/09/7209/1 -- To view, visit http://gerrit.cloudera.org:8080/7209 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I906465ad220236538175c80972ae055193f9bb45 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]>
