Adar Dembo 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 necessary. IIRC, a tserver sends a full tablet report while registering. Shouldn't that trigger the exact same behavior in the master as the previous (incremental) tablet report? 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? -- 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 <mpe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-HasComments: Yes