Bankim Bhavsar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17359 )

Change subject: [doc] KUDU-2181 Remove master doc changes
......................................................................


Patch Set 2:

(2 comments)

Combining all the multi-master doc changes in a single code review.
https://gerrit.cloudera.org/c/17352/5

http://gerrit.cloudera.org:8080/#/c/17359/2/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/17359/2/docs/administration.adoc@971
PS2, Line 971: . Stop process(es):
> I'm a little hesitant to document special steps for CM just yet, given I th
That's a good point. I'm removing CM specific steps.


http://gerrit.cloudera.org:8080/#/c/17359/2/docs/administration.adoc@975
PS2, Line 975: role(s).
> Aren't we only removing one at a time? Should this be singular?
Original guide allows for removal of multiple masters and I'm keeping it that 
way by asking user to to run  `kudu master remove` multiple times to remove 
multiple masters (See L980)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I58b584a1487dc0be4a2479aa80d1464f68d9fce5
Gerrit-Change-Number: 17359
Gerrit-PatchSet: 2
Gerrit-Owner: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 11 May 2021 22:59:24 +0000
Gerrit-HasComments: Yes

Reply via email to