Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13579 )

Change subject: [docs] Add 'CLI tools to access a cluster via cluster name' 
section
......................................................................


Patch Set 1:

(5 comments)

Thanks for doing this.

When submitting docs changes, we typically also push them to our own personal 
forks of Kudu on GitHub so that the contents of the AsciiDoc files are rendered 
for reviewers.

For example, here's a patch I wrote not too long ago to contributing.adoc: 
https://github.com/adembo/kudu/blob/docs/docs/contributing.adoc

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

http://gerrit.cloudera.org:8080/#/c/13579/1/docs/administration.adoc@1446
PS1, Line 1446: === CLI tools to access a cluster via cluster name
How about: Using cluster names in the `kudu` command line tool


http://gerrit.cloudera.org:8080/#/c/13579/1/docs/administration.adoc@1448
PS1, Line 1448: Kudu master rpc addresses are not easy to remember when use CLI 
tools, even
              : worse, when manage several Kudu clusters. It's able to use CLI 
tools to access
              : a Kudu cluster via cluster name alternatively.
Rewrite as: When using the `kudu` command line tool, it can be difficult to 
remember the precise list of Kudu master RPC addresses needed to communicate 
with a cluster, especially when managing multiple clusters. As an alternative, 
the command line tool can identify clusters by name. To use this functionality:


http://gerrit.cloudera.org:8080/#/c/13579/1/docs/administration.adoc@1452
PS1, Line 1452: . Make a Kudu configuration file directory, add an environment 
variable
              :   KUDU_CONFIG to this directoty.
              : . Add a new YAML format configuration file in this directoty, 
its content is like:
Rewrite:

- Create a new directory to store the Kudu configuration file.
- Export the path to this directory in the KUDU_CONFIG environment variable.
- Create a file called `kudurc` in the new directory.
- Populate `kudurc` as follows, substituting your own cluster names and RPC 
addresses:


http://gerrit.cloudera.org:8080/#/c/13579/1/docs/administration.adoc@1464
PS1, Line 1464: . Use CLI tools to access Kudu cluster via cluster name 
configured in the file above,
              :   starting with an extra character `@`, for the 
`master_addresses` section.
Rewrite: "When using the `kudu` command line tool, replace the list of Kudu 
master RPC addresses with the cluster name, prepended with the character `@`".


http://gerrit.cloudera.org:8080/#/c/13579/1/docs/administration.adoc@1475
PS1, Line 1475: NOTE: All CLI tools with a `master_addresses` section can be 
replaced by a cluster name.
Rewrite: "NOTE: Cluster names may be used as input in any invocation of the 
`kudu` command line tool that expects a list of Kudu master RPC addresses".



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib94a44b76008bc9d24e428000205e7c90e3ae653
Gerrit-Change-Number: 13579
Gerrit-PatchSet: 1
Gerrit-Owner: Yingchun Lai <405403...@qq.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
Gerrit-Reviewer: Yingchun Lai <405403...@qq.com>
Gerrit-Comment-Date: Wed, 12 Jun 2019 04:11:58 +0000
Gerrit-HasComments: Yes

Reply via email to