David Ribeiro Alves has posted comments on this change.

Change subject: docs: workflow for master migration
......................................................................


Patch Set 1:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/4300/1/docs/administration.adoc
File docs/administration.adoc:

Line 202: For true high availability and to avoid a single point of failure, 
Kudu clusters should be created
remove "true"


Line 236:   recovering from permanent master failures greatly, and is highly 
recommended. The alias should be
how? are you linking this to somewhere else, or are you adding this info on a 
later patch. If its the latter I would recommend adding this info there.


Line 244: * Identify and record the directory where the master's data will live.
this is confusing, the user can "choose" these now since it's running new 
daemons, what should be indentified and recorded?


Line 246: * Optional: configure a DNS cname or /etc/hsots alias to the master's 
hostname (e.g. `master-2`,
same as above


Line 282: ** `port` is the master's previously recorded RPC port number
it would be good to have examples for an actual correct command, maybe 
somewhere else too


Line 314: are working properly, consider performing the following sanity checks:
how does this validate that the new masters are working properly? if the user 
did stuff wrong nothing changed, wouldnt this work anyway?


-- 
To view, visit http://gerrit.cloudera.org:8080/4300
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9b9c66505e0efd1f4aef80884346507d4fe08d9c
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward #149
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to