Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13373 )
Change subject: [tool] update type of loadgen's --num_rows_per_thread flag ...................................................................... [tool] update type of loadgen's --num_rows_per_thread flag This patch updates the type of the --num_rows_per_thread command-line flag, changing it from unsigned. With that, the value of 0 is not a special value for this flag anymore. Instead, any negative value for this flag now means 'as many rows per thread as possible'. With this change, it's now possible to use the 'kudu perf loadgen' CLI tool to create empty tables, which is useful in various test scenarios. In addition, this patch contains a small clean-up on using the --keep_auto_table loadgen's flag in some test scenarios. Change-Id: I2712ac7678c9cfd9359629f11df3a86dd727997d Reviewed-on: http://gerrit.cloudera.org:8080/13373 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_perf.cc 3 files changed, 144 insertions(+), 101 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13373 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2712ac7678c9cfd9359629f11df3a86dd727997d Gerrit-Change-Number: 13373 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
