David Ribeiro Alves has posted comments on this change.

Change subject: docs: add master permanent failure recovery workflow
......................................................................


Patch Set 1:

(7 comments)

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

Line 288: * If using DNS aliases, override the empty value of the `Master 
Address` parameter for each role
you use a period in the line above, but an asterisk here. on purpose?


Line 362: important to replace the dead master; otherwise a second failure can 
lead to a loss of availability.
s/can/may also add to the end of the sentence: availability, depending on the 
number of configured/available masters. 
... or something like that.


Line 366: possible to replace a master if the deployment was created with DNS 
aliases. See the
on the other patch, you said "greatly help" but this seems to indicate there is 
no other way at all. seems like this should not be a hint and should be 
mandatory instead


Line 381:   workflow will refer to this master as the "recovery" master.
nit: maybe use "reference" instead? I find "recovery" ambiguous


Line 477: * Override the empty value of the `Master Address` parameter for the 
new role with the replacement
not sure what this means...


Line 485: . Restart the existing live masters. This results in a brief 
availability outage, but it should
You should mention this at the beginning, in bold letters.


Line 486:   last only as long as it takes for the masters to come back up.
you can add that masters usually take a very short while to come up.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I49d63efa76166bc548db75b0e43ae317c49f9e95
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: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to