David Ribeiro Alves has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/6191 Reviewed-by: Adar Dembo <[email protected]> Tested-by: David Ribeiro Alves <[email protected]> --- M src/kudu/integration-tests/delete_table-test.cc 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: David Ribeiro Alves: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/6191 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b5f21c718bad8c0b7112395082371f19a61507f Gerrit-PatchSet: 5 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: Todd Lipcon <[email protected]>
