Hello Anonymous Coward (314),

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/15018

to review the following change.


Change subject: [WIP] Normalize table names for Ranger
......................................................................

[WIP] 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-sensitive. 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.

Change-Id: I11431ff5bc75540edff56ef3d4ad384fa37d33d5
---
M CMakeLists.txt
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
6 files changed, 95 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/15018/1
--
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: newchange
Gerrit-Change-Id: I11431ff5bc75540edff56ef3d4ad384fa37d33d5
Gerrit-Change-Number: 15018
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Anonymous Coward (314)

Reply via email to