Hello Todd Lipcon,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: Make delete_table-test less flaky
......................................................................

Make delete_table-test less flaky

This patch reduces the flakiness of delete_table-test by solving
flakiness problems on two individual tests:

1. DeleteTableTest.TestAutoTombstoneAfterTabletCopyRemoteFails

This test triggers a remote crash that would sometimes prevent an RPC
response from arriving at the caller. We now account for this.

2. DeleteTableTombstonedParamTest.TestTabletTombstone

This test would sometimes not write quickly enough, resulting in a
timeout waiting for 3 wal files to appear on tablet 0. This could occur
when running under TSAN on a system under heavy load.

Now, the above two tests run successfully in 200/200 attempts on TSAN
under load.

This patch does not address the flakiness of
DeleteTableTest.TestDeleteTableWithConcurrentWrites yet. That test has
two issues that are being tracked as KUDU-1294 and KUDU-1797.

However a log message has been added to this commit to aid in debugging
that test in a later patch.

Change-Id: Iea7760009acc8a795225369721af97583974feba
---
M src/kudu/integration-tests/delete_table-test.cc
1 file changed, 22 insertions(+), 3 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iea7760009acc8a795225369721af97583974feba
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to