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

Reply via email to