Julian Hyde created CALCITE-4671:
------------------------------------
Summary: Allow Avatica connect-string properties in any case
(lowerCamel, UPPER_SNAKE, etc.)
Key: CALCITE-4671
URL: https://issues.apache.org/jira/browse/CALCITE-4671
Project: Calcite
Issue Type: Bug
Components: avatica
Reporter: Julian Hyde
Avatica's built-in properties are mostly lower_snake case (e.g.
{{truststore_password}}; the one exception being {{timeZone}} in lowerCamel;
see [client
reference|https://calcite.apache.org/avatica/docs/client_reference.html]);
Calcite's built-in properties are lowerCamel case (e.g.
[approximateTopN|https://calcite.apache.org/javadocAggregate/org/apache/calcite/config/CalciteConnectionProperty.html#APPROXIMATE_TOP_N];
see [Calcite JDBC driver
reference|https://calcite.apache.org/docs/adapter.html#jdbc-connect-string-parameters%C2%A0]).
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.)
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
