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 (#10).
Change subject: KUDU-2973 Add semi-database support for Ranger
......................................................................
KUDU-2973 Add semi-database support 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, 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 that works slightly different from Hive's
table name parser, as it allows non-ASCII characters and provides a
configurable default database name when the table name doesn't contain a
period.
Change-Id: I11431ff5bc75540edff56ef3d4ad384fa37d33d5
---
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
4 files changed, 109 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/15018/10
--
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: 10
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)
