Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/6534
Change subject: master: Retry background tasks even if TS UUID not registered ...................................................................... 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 --- 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, 101 insertions(+), 57 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6534/1 -- To view, visit http://gerrit.cloudera.org:8080/6534 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3a3de7fe87266f11392fd3bb0c74f19ad803de9d Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org>
