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

Jason Altekruse commented on DRILL-2304:
----------------------------------------

[~sudheeshkatkam] Without any strong opinions on the discussion thread about 
this I think we can safely go ahead and lowercase the names when they are 
registered. Might be worth logging a message to say that the name is being 
converted to lowercase whenever one is not already, this will keep us from 
completely ignoring the new options that are specified in the code with capital 
letters in their names.

> Case sensitivity - system and session options are case sensitive
> ----------------------------------------------------------------
>
>                 Key: DRILL-2304
>                 URL: https://issues.apache.org/jira/browse/DRILL-2304
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Storage - Information Schema
>    Affects Versions: 0.8.0
>            Reporter: Ramana Inukonda Nagaraj
>            Assignee: Sudheesh Katkam
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: DRILL-2304.1.patch.txt
>
>
> TBH I am not sure if this is a bug. 
> When trying to set a session option and I specify the name in a different 
> case the alter command fails. Considering the way we store session options 
> this might be an invalid bug but considering how typical Database hints and 
> options work this is a bug.
> {code}
> 0: jdbc:drill:> alter SESSION  set `STORE.PARQUET.COMPRESSION`='GZIP';
> Query failed: SetOptionException: Unknown option: STORE.PARQUET.COMPRESSION
> Error: exception while executing query: Failure while executing query. 
> (state=,code=0)
> {code}



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

Reply via email to