[CALCITE-4671] Allow Avatica connect-string properties in any case (lowerCamel, UPPER_SNAKE, etc.)

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: avatica
• Assignee: Unassigned
• Reporter: Julian Hyde
• Created: 28/Jun/21 23:27
• Updated: 28/Jun/21 23:27

Description

Avatica's built-in properties are mostly lower_snake case (e.g. truststore_password; the one exception being timeZone in lowerCamel; see client reference); Calcite's built-in properties are lowerCamel case (e.g. approximateTopN; see Calcite JDBC driver reference). Avatica's properties are inherited by Calcite, so there is bound to be confusion.

Avatica's connect string parser (also used in Calcite) should allow any case. Thus truststore_password, truststorePassword, TRUSTSTORE_PASSWORD, TruststorePassword are all acceptable. trustStorePassword is not acceptable, because "truststore" is one word.

In Calcite and Avatica doc, add a note that properties can are accepted in any case.

Change Avatica's properties to lowerCamel case in both doc and code. (The old forms, e.g. "truststore_password", will continue to work.)

Issue Links

• relates to: CALCITE-2322 Add fetch size support to connection url and JDBC statement