Hello 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 (#4).
Change subject: KUDU-2973 Normalize table names for Ranger
......................................................................
KUDU-2973 Normalize table names for Ranger
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.
Ranger's restrictions regarding identifiers are more lax, but we still
need to follow the stricter requirements as if a user turns on HMS
integration after Ranger authorization, the identifiers still need to be
valid and they must match in both places. The reason for this is that
HMS and its clients (Impala, Hive, Spark, etc) also perform
authorization so if a user is permitted to write to a table in Ranger,
they could still be denied when trying to access the same table through
Impala for example if the identifiers in HMS and Ranger didn't match up.
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/master/CMakeLists.txt
M src/kudu/master/catalog_manager.cc
A src/kudu/ranger/CMakeLists.txt
A src/kudu/ranger/ranger_client.cc
A src/kudu/ranger/ranger_client.h
9 files changed, 189 insertions(+), 19 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/15018/4
--
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: 4
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)
