Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16468 )
Change subject: tablet: add comment to summarize the bootstrap ...................................................................... Patch Set 1: Code-Review+2 (2 comments) Thank you for adding this! Just a couple of nits. http://gerrit.cloudera.org:8080/#/c/16468/1/src/kudu/tablet/tablet_bootstrap.cc File src/kudu/tablet/tablet_bootstrap.cc: http://gerrit.cloudera.org:8080/#/c/16468/1/src/kudu/tablet/tablet_bootstrap.cc@196 PS1, Line 196: per disk nit: to disk ? http://gerrit.cloudera.org:8080/#/c/16468/1/src/kudu/tablet/tablet_bootstrap.cc@200 PS1, Line 200: . tiny style nit here and elsewhere: either drop this period or start the item with a capital letter? -- To view, visit http://gerrit.cloudera.org:8080/16468 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I32d699efd97a3fdf57f47c30eded53f93f4553b7 Gerrit-Change-Number: 16468 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 17 Sep 2020 07:48:22 +0000 Gerrit-HasComments: Yes