[ 
https://issues.apache.org/jira/browse/KYLIN-2108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15589353#comment-15589353
 ] 

Julian Hyde commented on KYLIN-2108:
------------------------------------

That's a bit of a lame fix, isn't it. If will match 'DEFAULT.' if it occurs in 
character literals, which is wrong.

Is there are point in Kylin's SQL generation where you see each identifier? If 
an identifier is a reserved word, or if it contains punctuation, then you 
should quote it.

> Support usage of schema name "default" in SQL
> ---------------------------------------------
>
>                 Key: KYLIN-2108
>                 URL: https://issues.apache.org/jira/browse/KYLIN-2108
>             Project: Kylin
>          Issue Type: Improvement
>          Components: Query Engine
>            Reporter: Yifan Zhang
>            Assignee: Yifan Zhang
>            Priority: Minor
>             Fix For: v1.6.0
>
>         Attachments: KYLIN-2108.patch
>
>
> Calcite will treat "DEFAULT" as internal keyword, while "DEFAULT" is also 
> used as HIVE's default schema name. We need to escape such case of usage by 
> quote it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to