Hello Todd Lipcon, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6191
to look at the new patch set (#4).
Change subject: [delete_table-test] Don't fail on multiple attempts to delete
an orphaned tablet
......................................................................
[delete_table-test] Don't fail on multiple attempts to delete an orphaned tablet
TestUnknownTabletsAreNotDeleted is flaky with the following assertion:
/data/jenkins-workspace/kudu-workspace/src/kudu/integration-tests/delete_table-test.cc:1099:
Failure
Value of: num_delete_attempts
Actual: 3
Expected: 1
This is due to the master receiving more than 1 tablet report before
the tablet server has the time to delete the tablet. Security-related
changes likely shifted the timings and the heartbeat period is set
very low in this test, so this is more likely to happen. Nonetheless
the opportunity for flakyness was there even before these changes.
In addition to fixing the assertion on the metric, this patch also
adds an assertion that the tablet was actually deleted.
Change-Id: I2b5f21c718bad8c0b7112395082371f19a61507f
---
M src/kudu/integration-tests/delete_table-test.cc
1 file changed, 10 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/6191/4
--
To view, visit http://gerrit.cloudera.org:8080/6191
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2b5f21c718bad8c0b7112395082371f19a61507f
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
