[CALCITE-1263] Case-insensitive match and null default value for enum properties

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: avatica-1.9.0
• Component/s: avatica
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 29/May/16 05:43
• Updated: 31/Oct/16 04:16
• Resolved: 15/Jun/16 21:37

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 }.

Issue Links

• is depended upon by: CALCITE-1259 Allow connecting to a single schema without writing a model
• relates to: CALCITE-1270 Upgrade Calcite to avatica-1.9
• links to: PR