Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11467 )
Change subject: [hms] enable special characters support in mini HMS ...................................................................... Patch Set 2: > > > The catalog manager doesn't try to enforce Hive naming > conventions, > > > right? It just passes names through to the HMS for validation? > > > > Actually, catalog manager calls hms catalog for create/delete/alter > HMS tables, and hms catalog does validate the naming. Then HMS will > do another round of validation. > > Ah, you're referring to HmsCatalog::ParseTableName. I see the only > "special character" it allows is '/' (and perhaps '.'). Does an HMS > configured in this way allow other special characters that we still > forbid? No, currently the only special character supported is '/' in HMS ( https://github.com/apache/hive/blob/master/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java#L784). -- To view, visit http://gerrit.cloudera.org:8080/11467 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3ae5387c6c39a3d0c622316e5cda3236660806a4 Gerrit-Change-Number: 11467 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 19 Sep 2018 21:33:16 +0000 Gerrit-HasComments: No
