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