You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ha...@apache.org on 2018/09/19 21:46:58 UTC
[2/2] kudu git commit: [hms] enable special characters support in
mini HMS
[hms] enable special characters support in mini HMS
This commit explictly enables special characters support for mini HMS
which allows '/' in table names, to avoid test failure (e.g.
MasterHmsTest.TestUppercaseIdentifiers) caused by running with Hive
distributions that disable special characters support by default.
Change-Id: I3ae5387c6c39a3d0c622316e5cda3236660806a4
Reviewed-on: http://gerrit.cloudera.org:8080/11467
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/df93255d
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/df93255d
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/df93255d
Branch: refs/heads/master
Commit: df93255d9e0db86d797dabfb6e56dfc586c73f28
Parents: f3cd878
Author: Hao Hao <ha...@cloudera.com>
Authored: Tue Sep 18 18:40:23 2018 -0700
Committer: Hao Hao <ha...@cloudera.com>
Committed: Wed Sep 19 21:46:10 2018 +0000
----------------------------------------------------------------------
src/kudu/hms/mini_hms.cc | 9 +++++++++
1 file changed, 9 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/df93255d/src/kudu/hms/mini_hms.cc
----------------------------------------------------------------------
diff --git a/src/kudu/hms/mini_hms.cc b/src/kudu/hms/mini_hms.cc
index e27615a..cc7e6bc 100644
--- a/src/kudu/hms/mini_hms.cc
+++ b/src/kudu/hms/mini_hms.cc
@@ -221,6 +221,10 @@ Status MiniHms::CreateHiveSite() const {
//
// - hive.metastore.disallow.incompatible.col.type.changes
// Configures the HMS to allow altering and dropping columns.
+ //
+ // - hive.support.special.characters.tablename
+ // Configures the HMS to allow special characters such as '/' in table
+ // names.
static const string kFileTemplate = R"(
<configuration>
<property>
@@ -280,6 +284,11 @@ Status MiniHms::CreateHiveSite() const {
<name>hive.metastore.disallow.incompatible.col.type.changes</name>
<value>false</value>
</property>
+
+ <property>
+ <name>hive.support.special.characters.tablename</name>
+ <value>true</value>
+ </property>
</configuration>
)";