Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10591 )
Change subject: KUDU-1038 Deleting a tablet should also delete its log recovery directory, if any ...................................................................... Patch Set 10: (1 comment) Not sure if this is ready for re-review, I just took a quick look and it seems the review feedback has been addressed. If so please respond to the review feedback on a per-item basis. Thanks! http://gerrit.cloudera.org:8080/#/c/10591/10/src/kudu/consensus/log.h File src/kudu/consensus/log.h: http://gerrit.cloudera.org:8080/#/c/10591/10/src/kudu/consensus/log.h@145 PS10, Line 145: * nit: don't make these changes, keep the modifiers next to the type to be consistent w/ the rest of the Kudu code base -- To view, visit http://gerrit.cloudera.org:8080/10591 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If3f017caf41ac4e00bf89421d6e73924b9fdcbd0 Gerrit-Change-Number: 10591 Gerrit-PatchSet: 10 Gerrit-Owner: Anupama Gupta <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Fri, 08 Jun 2018 07:02:19 +0000 Gerrit-HasComments: Yes
