Dinesh Bhat has posted comments on this change. Change subject: [doc] 1.1.0 release notes for tools ......................................................................
Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/5012/3/docs/release_notes.adoc File docs/release_notes.adoc: PS3, Line 57: tablet > replica Done PS3, Line 59: delete the : replica of a tablet > "delete a replica" Done PS3, Line 60: when the tablet server is unable to restart due to bad state of the replica. > Is this part really necessary? "kudu local_replica delete" can be used for Yeah, I wanted to emphasize implicitly that this tool is mainly used when tablet server is offline for so and so reasons. Come to think of it, the fact that it's an action under 'local_replica' makes it obvious that this tool can be used only when the server is offline, isn't it ? Removed it now. Also, this tool only works with --clean_unsafe flag for the deletion to happen. otherwise it throws 'currently unsupported' and that support for 'tombstoning the tablet when server is offline' will be part of upcoming release. Although all this has clear help description under the tool, should we put that in release notes as well ? -- To view, visit http://gerrit.cloudera.org:8080/5012 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6dc80b356d4cec6d0065a515dce5fdd94581346a Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes