David Ribeiro Alves has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/6191

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, 12 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/6191/1
-- 
To view, visit http://gerrit.cloudera.org:8080/6191
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2b5f21c718bad8c0b7112395082371f19a61507f
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>

Reply via email to