[
https://issues.apache.org/jira/browse/CALCITE-1263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Josh Elser updated CALCITE-1263:
--------------------------------
Description:
In ConnectionConfigImpl, we allow properties based on enum classes. The getEnum
method throws "Required property 'name' not specified" if the default value is
null, but it should not; we should allow enum properties whose default value is
null.
Also, when resolving an enum property we should check the exact string first,
then look for case-insensitive matches. This will help if people write 'p=foo'
if 'p' is a property of type enum \{ FOO, BAZ \}.
was:
In ConnectionConfigImpl, we allow properties based on enum classes. The getEnum
method throws "Required property 'name' not specified" if the default value is
null, but it should not; we should allow enum properties whose default value is
null.
Also, when resolving an enum property we should check the exact string first,
then look for case-insensitive matches. This will help if people write 'p=foo'
if 'p' is a property of type enum { FOO, BAZ }.
> Case-insensitive match and null default value for enum properties
> -----------------------------------------------------------------
>
> Key: CALCITE-1263
> URL: https://issues.apache.org/jira/browse/CALCITE-1263
> Project: Calcite
> Issue Type: Bug
> Components: avatica
> Reporter: Julian Hyde
> Assignee: Julian Hyde
> Fix For: next
>
>
> In ConnectionConfigImpl, we allow properties based on enum classes. The
> getEnum method throws "Required property 'name' not specified" if the default
> value is null, but it should not; we should allow enum properties whose
> default value is null.
> Also, when resolving an enum property we should check the exact string first,
> then look for case-insensitive matches. This will help if people write
> 'p=foo' if 'p' is a property of type enum \{ FOO, BAZ \}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
