Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15307 )
Change subject: [tserver] fix error code for DeleteTablet requests ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15307/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15307/1//COMMIT_MSG@13 PS1, Line 13: If not do this, the tablet will not be deleted by tserver : until next time tserver send full tablet report to the leader master. Why is this a problem? The tserver will include the newly copied tablet in its report right after it finishes bootstrapping, right? At which point the master will respond by sending a new delete request? The issue with this patch is that it'll cause the master to retry the delete request over and over. Eventually it'll go through (when the tablet is done copying), but until then it's a waste of resources. -- To view, visit http://gerrit.cloudera.org:8080/15307 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I591dc8daceb81f9e800098be77c928d391cdc00a Gerrit-Change-Number: 15307 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 27 Feb 2020 17:10:15 +0000 Gerrit-HasComments: Yes
