[
https://issues.apache.org/jira/browse/SOLR-8593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15207685#comment-15207685
]
Joel Bernstein edited comment on SOLR-8593 at 3/23/16 1:29 AM:
---------------------------------------------------------------
Currently quoted identifiers do refer to columns. This was originally done
because Presto didn't support mixed case columns unless they were quoted. But
Presto fixed that problem. So the quoted identifiers as they are now don't
really serve a purpose. But I do believe that both Presto and Calcite allow
columns with quoted identifiers to support non parseable identifiers.
was (Author: joel.bernstein):
Currently quoted identifiers do refer to columns. This was originally done
because Presto didn't support mixed case columns unless they were quoted. But
Presto fixed that problem. So the quoted identifiers as they are now don't
really serve a purpose. But I do believe that both Presto and Calcite allow
columns for quoted identifiers to support non parseable identifiers.
> Integrate Apache Calcite into the SQLHandler
> --------------------------------------------
>
> Key: SOLR-8593
> URL: https://issues.apache.org/jira/browse/SOLR-8593
> Project: Solr
> Issue Type: Improvement
> Reporter: Joel Bernstein
> Fix For: master
>
>
> The Presto SQL Parser was perfect for phase one of the SQLHandler. It was
> nicely split off from the larger Presto project and it did everything that
> was needed for the initial implementation.
> Phase two of the SQL work though will require an optimizer. Here is where
> Apache Calcite comes into play. It has a battle tested cost based optimizer
> and has been integrated into Apache Drill and Hive.
> This work can begin in trunk following the 6.0 release. The final query plans
> will continue to be translated to Streaming API objects (TupleStreams), so
> continued work on the JDBC driver should plug in nicely with the Calcite work.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
