Alexey Serbin has posted comments on this change.

Change subject: docs: update installation with new OS support
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/4128/1//COMMIT_MSG
Commit Message:

Line 10: 1. The SLES 12, Jessie, and Xenial Cloudera repo files contain 
unsubstituted
nit: could you make the lines in the detailed description to be less than 72 
symbols long?


http://gerrit.cloudera.org:8080/#/c/4128/1/docs/installation.adoc
File docs/installation.adoc:

Line 113: provide the operating system commands to start and stop Kudu.
Nit: consider 'These packages provide configuration files along with init.d 
service scripts to manage Kudu processes.  Once these packages are installed, 
the corresponding Kudu processes are started/stopped automatically on OS 
start-up/shutdown.'.

Also, is it worth mentioning that these packages should not be installed if 
running the CM (ClouderaManager)?


PS1, Line 115: O
O --> o ?  I.e., why is the capital 'o' letter?  And in other places with the 
install section.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2f51b55561f18fbe28d6bf4ed507dfba65947dc0
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: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to