Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-8153

Support upper case and mixed case column identifiers in the SQL interface

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 6.0
    • Fix Version/s: 6.0
    • Component/s: SolrJ
    • Labels:
      None

      Description

      The version of the Presto parser currently in Solr is lower casing all SQL identifiers unless they are string literals (single quotes). This appears to be happening in the QualifiedName class.

      The latest version of the Presto parser has changed the QualifiedName class to maintain the original casing. This will allow Solr to maintain the original case of SQL identifiers without requiring quoted identifiers.

      1. SOLR-8153.patch
        41 kB
        Joel Bernstein

        Issue Links

          Activity

          Hide
          joel.bernstein Joel Bernstein added a comment -

          Patch with new version of the Presto parser and changes in the SQLHandler to support upper case and mixed case columns.

          Show
          joel.bernstein Joel Bernstein added a comment - Patch with new version of the Presto parser and changes in the SQLHandler to support upper case and mixed case columns.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 1708259 from Joel Bernstein in branch 'dev/trunk'
          [ https://svn.apache.org/r1708259 ]

          SOLR-8153: Support upper case and mixed case column identifiers in the SQL interface

          Show
          jira-bot ASF subversion and git services added a comment - Commit 1708259 from Joel Bernstein in branch 'dev/trunk' [ https://svn.apache.org/r1708259 ] SOLR-8153 : Support upper case and mixed case column identifiers in the SQL interface

            People

            • Assignee:
              joel.bernstein Joel Bernstein
              Reporter:
              joel.bernstein Joel Bernstein
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development