Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21717 )
Change subject: KUDU-3606 [tools] Show column ids when using 'kudu table describe' ...................................................................... KUDU-3606 [tools] Show column ids when using 'kudu table describe' Column id is an internal concept, this patch adds a new flag --show_column_id to indicate whether to show it when using 'kudu table describe'. It's helpful to compare whether two tables' schema are competely the same with considering the internal column id as well. Change-Id: Ib9b28bd8f879bb6cace1683bc366c72772caebdc Reviewed-on: http://gerrit.cloudera.org:8080/21717 Reviewed-by: Yifan Zhang <[email protected]> Reviewed-by: Wang Xixu <[email protected]> Tested-by: Yingchun Lai <[email protected]> Reviewed-by: Marton Greber <[email protected]> --- M src/kudu/client/schema.cc M src/kudu/tools/kudu-admin-test.cc M src/kudu/tools/tool_action_table.cc 3 files changed, 82 insertions(+), 37 deletions(-) Approvals: Yifan Zhang: Looks good to me, approved Wang Xixu: Looks good to me, but someone else must approve Yingchun Lai: Verified Marton Greber: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/21717 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib9b28bd8f879bb6cace1683bc366c72772caebdc Gerrit-Change-Number: 21717 Gerrit-PatchSet: 3 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
