Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-4671

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

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: avatica
    • Labels:
      None

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

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                julianhyde Julian Hyde
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: