Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8133 )
Change subject: [tablet copy] end session before bootstrapping tablet ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8133/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/8133/1//COMMIT_MSG@12 PS1, Line 12: Prior to this patch, the tablet copy sessions were ended only after : the tablet start-up process completed. However, there isn't any : explicit constraint which requires that behavior. > I thought we did this to make sure that the leader (of whomever the replica I'm curious why is that? I thought once all the data and logs are copied, we can delete the source, if it's scheduled for deletion. Why would we need to postpone that moment till the new tablet is started? If it fails, it would be necessary to start a new tablet copy session, wouldn't it? -- To view, visit http://gerrit.cloudera.org:8080/8133 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I20592007a40113d8409f121b226bcccca14e8300 Gerrit-Change-Number: 8133 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Comment-Date: Mon, 25 Sep 2017 20:13:18 +0000 Gerrit-HasComments: Yes
