Mike Percy has submitted this change and it was merged. Change subject: KUDU-1991. master: Retry background tasks even if TS UUID not registered ......................................................................
KUDU-1991. master: Retry background tasks even if TS UUID not registered Before this patch, if a master had just started up and got a message from a tablet server that would require some action in response (for example, deleting an evicted replica or replacing a replica in an under-replicated tablet) then if an affected tablet server had not registered with the master the task would fail and would not be retried. This patch causes the task to be retried even if the initial tablet server UUID lookup fails. This change is tested by AdminCliTest.TestUnsafeChangeConfigOnSingleFollower, which prior to this patch was flaky (7/200 tests failed on dist-test before the patch). After the changes to that test, it requires the above master fix to pass but is no longer flaky (200/200 passed on dist-test). Change-Id: I3a3de7fe87266f11392fd3bb0c74f19ad803de9d Reviewed-on: http://gerrit.cloudera.org:8080/6534 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Mike Percy <mpe...@apache.org> --- M src/kudu/integration-tests/cluster_itest_util.cc M src/kudu/integration-tests/cluster_itest_util.h M src/kudu/master/catalog_manager.cc M src/kudu/tools/kudu-admin-test.cc 4 files changed, 102 insertions(+), 61 deletions(-) Approvals: Mike Percy: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/6534 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a3de7fe87266f11392fd3bb0c74f19ad803de9d Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot