Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/19263 )
Change subject: KUDU-3408 Better "kudurc" documentation ...................................................................... Patch Set 2: Code-Review+1 (2 comments) Thank you for the patch. http://gerrit.cloudera.org:8080/#/c/19263/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19263/2//COMMIT_MSG@13 PS2, Line 13: adds a reference to this feature Since you are modifying the doc to provide useful tips for users, maybe mention one more alternative: set an environment variable in the current shell session and use it later on to substitute Kudu master addresses, e.g.: $ export M=addr1,addr2,addr3 $ kudu table list $M http://gerrit.cloudera.org:8080/#/c/19263/2/docs/quickstart.adoc File docs/quickstart.adoc: http://gerrit.cloudera.org:8080/#/c/19263/2/docs/quickstart.adoc@126 PS2, Line 126: can identify clusters by name nit: maybe, use this text for the #using_cluster_names_in_kudu_tool link instead of adding extra text which tries to duplicate the name of the section? -- To view, visit http://gerrit.cloudera.org:8080/19263 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I33d9a6d579343ff7333af955bf309ac66bfb0c7b Gerrit-Change-Number: 19263 Gerrit-PatchSet: 2 Gerrit-Owner: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Mon, 28 Nov 2022 21:54:30 +0000 Gerrit-HasComments: Yes
