Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14718 )

Change subject: docs: add docs for maintenance mode
......................................................................


Patch Set 2: Code-Review+1

(8 comments)

A few nits.

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

http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1468
PS2, Line 1468: maintenance
So, is it 'maintenance' or 'maintenance mode', or 'maintenance state'?


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1469
PS2, Line 1469: will not be
nit: are not

I think that using the present tense simplifies this sentence.


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1472
PS2, Line 1472: a
its


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1473
PS2, Line 1473: will be
is


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1485
PS2, Line 1485: will show up
nit: is shown


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1486
PS2, Line 1486: ksck
kudu cluster ksck


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1486
PS2, Line 1486: leader
drop


http://gerrit.cloudera.org:8080/#/c/14718/2/docs/administration.adoc@1487
PS2, Line 1487: the following can be run
nit: run the following



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I36b9eddc1d4d4a4e4cb149058fc6d6f438e47f1f
Gerrit-Change-Number: 14718
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Priyanka Chheda <pchh...@cloudera.com>
Gerrit-Comment-Date: Sat, 16 Nov 2019 02:23:02 +0000
Gerrit-HasComments: Yes

Reply via email to