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/src/kudu/tserver/ts_tablet_manager.cc File src/kudu/tserver/ts_tablet_manager.cc: http://gerrit.cloudera.org:8080/#/c/8133/1/src/kudu/tserver/ts_tablet_manager.cc@a629 PS1, Line 629: : > We likely need an explanation here that states why this needs to survive un I think it's a good idea. Let me dig in a bit to get a better understanding and then I'll update this changelist as needed (hope to do that later this evening). -- 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 21:39:00 +0000 Gerrit-HasComments: Yes