Alexey Serbin has posted comments on this change. Change subject: [doc] mention the loadgen tool in the release notes ......................................................................
Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5020/1/docs/release_notes.adoc File docs/release_notes.adoc: PS1, Line 64: * The `insert-generated-rows` tool has been removed. > Was this a part of CLI earlier ? if not, I think the mentioning of this bei That was a stand-alone binary outside of the 'kudu' umbrella tools. ok, will update. PS1, Line 65: new > probably redundant. Done PS1, Line 66: temporary > may be s/temporary/new ? Not sure that would bring clarity here. The idea was to mention that the tool can run either against temporary tables which it creates itself or against existing ones. Probably, I should drop 'already' or all the sentence altogether if it's so confusing. -- To view, visit http://gerrit.cloudera.org:8080/5020 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1a227c475b05cb394718812ae9c825022f67994d Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dinesh Bhat <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
