kudu-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Attila Bukor (Code Review)" <ger...@cloudera.org>
Subject [kudu-CR] KUDU-3090: Native owner metadata in Kudu
Date Tue, 30 Jun 2020 14:40:40 GMT
Attila Bukor has uploaded a new patch set (#4) to the change originally created by Grant Henke.
( http://gerrit.cloudera.org:8080/15841 )

Change subject: KUDU-3090: Native owner metadata in Kudu
......................................................................

KUDU-3090: Native owner metadata in Kudu

Apache Sentry and Apache Ranger both support permissions granted to
table owners, but as Sentry integrates with Apache Hive Metastore (HMS)
and stores its metadata in it, Kudu didn't need to store table ownership
to support granting permissions to owners.

Apache Ranger on the other hand doesn't depend on HMS and needs Kudu to
tell it if the owner is attempting to authorize an action, so to enable
users to grant privileges to owners we need to support ownership
natively.

This patch adds the basic plumbing for table ownership, synchronizing
ownership metadata with HMS both using the notification log listener and
via tooling, and setting the owner on CREATE TABLE and ALTER TABLE
requests in the C++ client.

Supporting this in the Java and Python clients, authorizing these
requests, and support for ownership in authorization will come in
follow-up patches.

Credit goes to Grant Henke <granthenke@apache.org> for the initial
version of this patch.

Design doc: https://s.apache.org/kudu-ownership-design

Change-Id: I67f5bfdf56d409960365fd5803913a2d3800831d
---
M src/kudu/client/client-internal.cc
M src/kudu/client/client-internal.h
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/table-internal.cc
M src/kudu/client/table-internal.h
M src/kudu/client/table_alterer-internal.cc
M src/kudu/client/table_alterer-internal.h
M src/kudu/client/table_creator-internal.h
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/hms_itest-base.cc
M src/kudu/integration-tests/hms_itest-base.h
M src/kudu/integration-tests/master_hms-itest.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
M src/kudu/master/hms_notification_log_listener.cc
M src/kudu/master/master.proto
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_hms.cc
21 files changed, 292 insertions(+), 60 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/15841/4
-- 
To view, visit http://gerrit.cloudera.org:8080/15841
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I67f5bfdf56d409960365fd5803913a2d3800831d
Gerrit-Change-Number: 15841
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke <granthenke@apache.org>
Gerrit-Reviewer: Attila Bukor <abukor@apache.org>
Gerrit-Reviewer: Grant Henke <granthenke@apache.org>
Gerrit-Reviewer: Hao Hao <hao.hao@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

Mime
  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message