Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18446 )
Change subject: [tools] Add '--create_table_replication_factor' flag for 'kudu table copy' ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/18446/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18446/1//COMMIT_MSG@9 PS1, Line 9: Now '--create_table_replication_factor' flag is added to set the : replication factor of the destination table if the table will be : created. nit: how about Now it's possible to specify the replication factor for the destination table when copying a table. http://gerrit.cloudera.org:8080/#/c/18446/1/src/kudu/tools/tool_action_table.cc File src/kudu/tools/tool_action_table.cc: PS1: Please add corresponding test scenario into kudu-tool-test.cc http://gerrit.cloudera.org:8080/#/c/18446/1/src/kudu/tools/tool_action_table.cc@1555 PS1, Line 1555: .AddOptionalParameter("create_table") : .AddOptionalParameter("create_table_replication_factor") Is it possible to specify --create_table_replication_factor when not specifying --create_table? What's the expected behavior in such a case? -- To view, visit http://gerrit.cloudera.org:8080/18446 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9a4eebdcf85b5ec3666e023194b8c06d66b0a683 Gerrit-Change-Number: 18446 Gerrit-PatchSet: 1 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 25 Apr 2022 18:25:00 +0000 Gerrit-HasComments: Yes
