Mike Percy has posted comments on this change. Change subject: master: Retry background tasks even if TS UUID not registered ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6534/1//COMMIT_MSG Commit Message: PS1, Line 14: This patch causes the task to be retried even if the initial tablet : server UUID lookup fails. > I'm not necessarily opposed to this patch, but I don't understand why it's SendDeleteReplicaRequest() is called as part of CatalogManager::HandleReportedTablet() so it's only guaranteed to be a one-time thing to get a full tablet report that includes this tablet from this tablet server. http://gerrit.cloudera.org:8080/#/c/6534/1/src/kudu/integration-tests/cluster_itest_util.h File src/kudu/integration-tests/cluster_itest_util.h: Line 188: > Unintended addition? Done -- To view, visit http://gerrit.cloudera.org:8080/6534 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3a3de7fe87266f11392fd3bb0c74f19ad803de9d Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-HasComments: Yes
