Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8567 )
Change subject: catalog_manager_tsk-itest: ensure that test eventually makes progress ...................................................................... catalog_manager_tsk-itest: ensure that test eventually makes progress This test previously tried to introduce a lot of master leader elections by setting a very low heartbeat and failure interval. This worked, but sometimes worked so well that the test never made progress and couldn't obtain a stable leader long enough to create a table. This patch changes the test to instead use a separate thread which triggers elections manually on all the leaders. The elections start off very frequent and then back off as the test progresses to ensure that by the end, the leaders do actually make progress. I verified that this still covers the case of a failed write when writing TSKs by changing the RETURN_NOT_OK to a CHECK_OK when storing the TSK. With the CHECK_OK, the test failed nearly immediately. Change-Id: I3ecda0c269225e7674bc384fee652576b110ae7b Reviewed-on: http://gerrit.cloudera.org:8080/8567 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- M src/kudu/integration-tests/catalog_manager_tsk-itest.cc 1 file changed, 46 insertions(+), 20 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8567 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3ecda0c269225e7674bc384fee652576b110ae7b Gerrit-Change-Number: 8567 Gerrit-PatchSet: 3 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>