David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5349
Change subject: Add integration tests for duplicate keys ...................................................................... Add integration tests for duplicate keys This splits the "crashy nodes" and "churny elections" test of raft_consensus-itest into unique and duplicate key variants. This change is meant to stress any possible deadlock scenarios related to transaction commit/abort and 2-phase locking for which we didn't have much coverage. Additionally this also disallows timeouts on writes and requires an exact count of the rows at the end. This is now possible due to exactly once semantics. Finally this changes the cluster verifier to use snapshot scans and changes the timeout of another scan in that test. These two changes deflaked the test from 27/1000 to 3/1000 with asan, slow mode, and 1 stress thread (any more and the test becomes much more flaky, as before). Change-Id: I6d018281d412ae034bd7b70c8311077a52b2795d --- M src/kudu/integration-tests/cluster_verifier.cc M src/kudu/integration-tests/raft_consensus-itest.cc 2 files changed, 122 insertions(+), 65 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/5349/1 -- To view, visit http://gerrit.cloudera.org:8080/5349 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6d018281d412ae034bd7b70c8311077a52b2795d Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org>