Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21695 )
Change subject: KUDU-3606 [tools] Check column IDs when --strict_column_id is enabled ...................................................................... KUDU-3606 [tools] Check column IDs when --strict_column_id is enabled This patch ensures the destination table has the same user-facing schema and internal column ids as the source table when using 'kudu table copy'. If they are different, more details, including column ids, default values, will be shown in error messages. A new flag --strict_column_id is added to indicate whether to compare column ids when checking if the schemas match. Change-Id: I77af7b92b45f3866cc8b699e61b9e71b73ed6c4b Reviewed-on: http://gerrit.cloudera.org:8080/21695 Tested-by: Yingchun Lai <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/table_scanner.cc M src/kudu/tools/tool_action_table.cc 3 files changed, 183 insertions(+), 32 deletions(-) Approvals: Yingchun Lai: Verified Mahesh Reddy: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/21695 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I77af7b92b45f3866cc8b699e61b9e71b73ed6c4b Gerrit-Change-Number: 21695 Gerrit-PatchSet: 8 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
