Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9255 )
Change subject: KUDU-2274 [itest] stress test for replica replacement ...................................................................... Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/9255/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/9255/5//COMMIT_MSG@11 PS5, Line 11: ar > are Done http://gerrit.cloudera.org:8080/#/c/9255/5/src/kudu/integration-tests/raft_consensus_stress-itest.cc File src/kudu/integration-tests/raft_consensus_stress-itest.cc: http://gerrit.cloudera.org:8080/#/c/9255/5/src/kudu/integration-tests/raft_consensus_stress-itest.cc@236 PS5, Line 236: > nit: why not iteration++ here? Because I don't want to count retry due to a ksck failure as an iteration: if RunKsck() fails, the logic starts another loop, but that case is not counted as 'iteration' in sense of this scenario. http://gerrit.cloudera.org:8080/#/c/9255/5/src/kudu/integration-tests/raft_consensus_stress-itest.cc@277 PS5, Line 277: if (ksck_failures_in_a_row > FLAGS_test_max_ksck_failures) { : // Suspecting a Raft consensus failure while running ksck with shorter : // timeout (see above). Run an extra round of ksck with the regular timeout : // to verify that replicas haven't really converged and, if so, just bail : // right at this point. : const auto& s = v.RunKsck(); : if (!s.ok()) { : FAIL() << Substitute("$0: tablet replicas haven't converged", s.ToString()); : } : } > isn't this the same as line 288 where we run v.CheckCluster() ? Yes, CheckCluster contains RunKsck(). However, it's wrapped into some extra logic for retries. Here I wanted to short-circuit and report an error right away if the ksck has already failed too many times in a row. -- To view, visit http://gerrit.cloudera.org:8080/9255 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I036a882f7e9132a5c26013227c50c0699b59ed6e Gerrit-Change-Number: 9255 Gerrit-PatchSet: 6 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Tue, 20 Feb 2018 23:54:32 +0000 Gerrit-HasComments: Yes
