David Ribeiro Alves has submitted this change and it was merged. Change subject: Add snapshot scans to fuzz-itest ......................................................................
Add snapshot scans to fuzz-itest This adds a new operation to fuzz-itest: snapshot scans at a timestamp. When generating random operations, scans at a timestamp are now in the mix. The generator records how timestamps will progress with writes and is careful to make sure that scans happen in the past. After the test case is generated, but before it is run, the timestamps for the scans are deduplicated and sorted to that we save the state immediately before those (and only those) timestamps. This would fail very often before the REINSERT patches but passes with it. Ran 500 loops of fuzz-itest in dist-test in ASAN with: - KUDU_ALLOW_SLOW_TESTS=1 - --stress_cpu_threads=4 All tests passed. Result: http://dist-test.cloudera.org//job?job_id=david.alves.1480096588.30319 Change-Id: I4d15129e83c5c9afa9b643e674c8a23e18a2fa0e Reviewed-on: http://gerrit.cloudera.org:8080/4996 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- M src/kudu/integration-tests/fuzz-itest.cc M src/kudu/tablet/key_value_test_schema.h M src/kudu/tserver/tablet_service.cc 3 files changed, 206 insertions(+), 48 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4996 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4d15129e83c5c9afa9b643e674c8a23e18a2fa0e Gerrit-PatchSet: 17 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>