Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15204 )
Change subject: [examples] Add a quickstart guide for YCSB ...................................................................... Patch Set 2: (3 comments) Thank you for adding this new document! http://gerrit.cloudera.org:8080/#/c/15204/2/examples/quickstart/ycsb/README.adoc File examples/quickstart/ycsb/README.adoc: http://gerrit.cloudera.org:8080/#/c/15204/2/examples/quickstart/ycsb/README.adoc@30 PS2, Line 30: Maybe, add another option to build Kudu from source and then use $KUDU_ROOT/src/kudu/scripts/start_kudu.sh scripts to start Kudu cluster? http://gerrit.cloudera.org:8080/#/c/15204/2/examples/quickstart/ycsb/README.adoc@38 PS2, Line 38: -O --location nit: maybe, use the same option notation for both flags? I.e., either '-LO' (short notation) or '--location --remote-name' http://gerrit.cloudera.org:8080/#/c/15204/2/examples/quickstart/ycsb/README.adoc@39 PS2, Line 39: xfvz nit: xfv (drop 'z' since it's not needed here) -- To view, visit http://gerrit.cloudera.org:8080/15204 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I26255c9389a8bb59af1009cd67f720149e59f4a1 Gerrit-Change-Number: 15204 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 11 Feb 2020 18:50:43 +0000 Gerrit-HasComments: Yes
