Hello Tidy Bot, Kudu Jenkins, Anonymous Coward (314), Adar Dembo, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15018
to look at the new patch set (#7).
Change subject: KUDU-2973 Refactor table names in catalog manager
......................................................................
KUDU-2973 Refactor table names in catalog manager
The table names are "normalized" if HMS integration is enabled so that
they can be synchronized to HMS. Hive's table/database identifiers are
pretty restrictive (alphanumerical ASCII characters + underscore and
forward slash) and it's case-insensitive. As Kudu doesn't support
databases it imposes a further restriction: the table identifiers to be
synchronized with HMS have to contain exactly one "." character which
separates the database name from the table name in HMS.
Catalog manager used to normalize the table name when HMS integration is
enabled and use that for authorization among other things.
Ranger's restrictions regarding identifiers are more lax, can be case
sensitive and is not limited to ASCII.
This commit introduces a method to parse database and table name for
Ranger from a table identifier and refactors catalog manager to use the
original table name for authorization instead of the normalized one and
push the normalization to Sentry's implementation instead.
Change-Id: I11431ff5bc75540edff56ef3d4ad384fa37d33d5
---
M CMakeLists.txt
M src/kudu/common/table_util-test.cc
M src/kudu/common/table_util.cc
M src/kudu/common/table_util.h
M src/kudu/integration-tests/master_hms-itest.cc
M src/kudu/master/CMakeLists.txt
M src/kudu/master/catalog_manager.cc
M src/kudu/master/sentry_authz_provider.cc
A src/kudu/ranger/CMakeLists.txt
A src/kudu/ranger/ranger_client.cc
A src/kudu/ranger/ranger_client.h
11 files changed, 209 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/15018/7
--
To view, visit http://gerrit.cloudera.org:8080/15018
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I11431ff5bc75540edff56ef3d4ad384fa37d33d5
Gerrit-Change-Number: 15018
Gerrit-PatchSet: 7
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Anonymous Coward (314)
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
