Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15018 )
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 (will not be used for Kudu) 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 Reviewed-on: http://gerrit.cloudera.org:8080/15018 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> Reviewed-by: Hao Hao <[email protected]> --- 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, 152 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, but someone else must approve Hao Hao: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I11431ff5bc75540edff56ef3d4ad384fa37d33d5 Gerrit-Change-Number: 15018 Gerrit-PatchSet: 14 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
