Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14958 )
Change subject: [tools] Add 'run_cleanup' option for 'kudu perf loadgen' ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/14958/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14958/2//COMMIT_MSG@9 PS2, Line 9: Add 'run_cleanup' option to provide a way to cleanup test data written to : the table, especially an existing user table. But deletions in Kudu just pile on additional load unless their rowsets are compacted (KUDU-859). Aren't we better off just dropping the modified table/partition entirely? http://gerrit.cloudera.org:8080/#/c/14958/2/src/kudu/tools/tool_action_perf.cc File src/kudu/tools/tool_action_perf.cc: http://gerrit.cloudera.org:8080/#/c/14958/2/src/kudu/tools/tool_action_perf.cc@307 PS2, Line 307: DEFINE_bool(run_cleanup, true, Why should this default to true? -- To view, visit http://gerrit.cloudera.org:8080/14958 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1e75adde434bac5e88151361655526b91f327b4c Gerrit-Change-Number: 14958 Gerrit-PatchSet: 2 Gerrit-Owner: Yingchun Lai <405403...@qq.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sun, 29 Dec 2019 18:45:05 +0000 Gerrit-HasComments: Yes