[jira] [Commented] (SOLR-9146) Parallel SQL engine should support >, >=, <, <= syntax

["Kevin Risden (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SOLR-9146?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15552116#comment-15552116 ] 

Kevin Risden commented on SOLR-9146:
------------------------------------

I had forgotten about this and finally had time to look at this closer. I am currently testing a patch that will at least make this start to work. The better fix would be to get the integration of Calcite in (SOLR-8593) so we can rely on that for parsing things we don't natively support.

> Parallel SQL engine should support >, >=, <, <= syntax
> ------------------------------------------------------
>
>                 Key: SOLR-9146
>                 URL: https://issues.apache.org/jira/browse/SOLR-9146
>             Project: Solr
>          Issue Type: New Feature
>          Components: Parallell SQL
>            Reporter: Timothy Potter
>            Assignee: Kevin Risden
>
> this gives expected result:
>      SELECT title_s, COUNT(*) as cnt
>         FROM movielens
>      WHERE genre_ss='action' AND rating_i='[4 TO 5]'
> GROUP BY title_s
> ORDER BY cnt desc
>          LIMIT 5
> but using >= 4 doesn't give same results (my ratings are 1-5):
>       SELECT title_s, COUNT(*) as cnt
>          FROM movielens
>       WHERE genre_ss='action' AND rating_i >= 4
> GROUP BY title_s
> ORDER BY cnt desc
>           LIMIT 5
> on the Solr side, I see queries forumlated as:
> 2016-05-21 14:53:43.096 INFO  (qtp1435804085-1419) [c:movielens
> s:shard1 r:core_node1 x:movielens_shard1_replica1] o.a.s.c.S.Request
> [movielens_shard1_replica1]  webapp=/solr path=/export
> params={q=((genre_ss:"action")+AND+(rating_i:"4"))&distrib=false&fl=title_s&sort=title_s+desc&wt=json&version=2.2}
> hits=2044 status=0 QTime=0
> which is obviously wrong ... 
> In general, rather than crafting an incorrect query that gives the
> wrong results, we should throw an exception stating that the syntax is
> not supported.
> Also, the ref guide should be updated to contain a known limitations section so users don't have to guess at what SQL features are supported by Solr.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org