Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11398 )
Change subject: HMS integration: set table owner field in HMS table metadata ...................................................................... HMS integration: set table owner field in HMS table metadata Other systems use table ownership for purposes like assigning privileges. This patch sets the owner field in the HMS for Kudu tables to the client's user name. A follow-up patch will add additional APIs to the client CreateTable builders which will allow clients to override the owner, for situations in which the client is actually proxying through the table creation on behalf of a different user. The HMS fix tool will create HMS table metadata for Kudu tables which are missing it. This replacement table metadata will omit the table owner, since it can't be reconstructed just from the Kudu table metadata. This is the conservative approach, if we want to extend the tool to allow passing in the owner as a flag we can do that in the future. Change-Id: I1a25aa0bb52bdd28df28a078fe91f55db9e29482 Reviewed-on: http://gerrit.cloudera.org:8080/11398 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <danburk...@apache.org> --- M src/kudu/hms/hms_catalog-test.cc M src/kudu/hms/hms_catalog.cc M src/kudu/hms/hms_catalog.h M src/kudu/integration-tests/master_hms-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_hms.cc 7 files changed, 145 insertions(+), 55 deletions(-) Approvals: Kudu Jenkins: Verified Dan Burkert: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11398 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1a25aa0bb52bdd28df28a078fe91f55db9e29482 Gerrit-Change-Number: 11398 Gerrit-PatchSet: 8 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot