Todd Lipcon has submitted this change and it was merged. Change subject: Fix bug in incorrect response rebuilding on tablet bootstrap ......................................................................
Fix bug in incorrect response rebuilding on tablet bootstrap This fixes the bug described in: https://gerrit.cloudera.org/#/c/5417/ ... and enables the test disabled in that patch. Along the way this also performs some cleanup of tablet bootstrap. I ran raft_consensus-itest's TestInsertDuplicateKeysWithCrashyNodes on dist-test for 5000 loops with the following config: KUDU_ALLOW_SLOW_TESTS=1 ../../build-support/dist_test.py \ --disable-sharding loop -n 5000 -- bin/raft_consensus-itest \ --gtest_filter=*DuplicateKeysWithCrashyNodes Prior to this patch the test failed 62/5000: http://dist-test.cloudera.org//job?job_id=david.alves.1493915326.763 After this patch the test passes 5000/5000: http://dist-test.cloudera.org//job?job_id=david.alves.1493914745.27867 Change-Id: I1219ed5f7835e93cd7f3b128cedd650fc3384482 Reviewed-on: http://gerrit.cloudera.org:8080/5489 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/tablet/row_op.cc M src/kudu/tablet/row_op.h M src/kudu/tablet/tablet.proto M src/kudu/tablet/tablet_bootstrap.cc M src/kudu/tserver/tablet_server-test.cc 5 files changed, 122 insertions(+), 114 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5489 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1219ed5f7835e93cd7f3b128cedd650fc3384482 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
