kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ha...@apache.org
Subject [2/2] kudu git commit: [hms] enable special characters support in mini HMS
Date Wed, 19 Sep 2018 21:46:58 GMT
[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 <adar@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 <hao.hao@cloudera.com>
Authored: Tue Sep 18 18:40:23 2018 -0700
Committer: Hao Hao <hao.hao@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>
   )";
 


Mime
View raw message